Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Docs with vitepress #136

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

emersonbottero
Copy link

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

I am planning to deploy this via Azure (same as xCAD). I believe this is for GitHub Actions. Could you please remove this from the commit?

Copy link
Contributor

@artem1t artem1t left a comment

Choose a reason for hiding this comment

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

Thanks a lot, this is a great job! I have added few comments. Basically, those are only related to the changes outside of the docs folder. To keep the PR clean, could you please revert any changes outside of docs folder (e.g. build, samples, src, templates, tests) and all the Github action files and only keep the changes in the docs folder?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think .gitignore are mainly kept as local files (not in the repo). Could you please remove from the repo?

Copy link
Author

Choose a reason for hiding this comment

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

The .gitignore is meant to prevent committing unwanted files in the repo.. for .net would be the obj and bin folders for the docs it would be the dist folder and node_modules.

Copy link
Contributor

Choose a reason for hiding this comment

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

README.md file is only used for the repo front page and NuGet (e.g. not run via docs). Could you please revert to the previous version?

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this file used for GitHub actions?

Copy link
Author

Choose a reason for hiding this comment

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

no.. it is used to restore packages used in the docs.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this file used for the GitHub actions?

Copy link
Author

Choose a reason for hiding this comment

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

it keeps the version of each package used so in the cicd you are sure it install the especific versions.

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