Skip to content

docs: add git submodule handling documentation#82

Merged
alirezanet merged 3 commits intoalirezanet:masterfrom
Tychus:Doc/SubmoduleDocumentation
Aug 26, 2023
Merged

docs: add git submodule handling documentation#82
alirezanet merged 3 commits intoalirezanet:masterfrom
Tychus:Doc/SubmoduleDocumentation

Conversation

@Tychus
Copy link
Copy Markdown
Contributor

@Tychus Tychus commented Aug 20, 2023

Documentation for PR:81

With the global installation, you don't need to add the `dotnet` prefix to the commands.
:::

## Husky and git submodules
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think submodule related docs don't belong to the getting-started page since it might confuse the new users. It would be nice if you move it to a separate menu/file.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point i'll see what i can do

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alirezanet i'm having some issues running the doc locally, running the commands below starts VuePress and all but when i open any link it's a blank page.

npm install
npm run dev

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is probably because I was using the beta version of vuepress ... I'll look into this later if you couldn't fix it. 👌

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Tychus,

I solved the problem, you're able to run the docs locally now.
just remove the node_modules folder before running it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hope this is clear enough.

Copy link
Copy Markdown
Owner

@alirezanet alirezanet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, 🚀💐

@alirezanet alirezanet merged commit bfae1ab into alirezanet:master Aug 26, 2023
@Tychus Tychus deleted the Doc/SubmoduleDocumentation branch September 2, 2023 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants