Skip to content

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