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

Merge from github/gitignore #2

Merged
merged 6 commits into from
Oct 18, 2024
Merged

Merge from github/gitignore #2

merged 6 commits into from
Oct 18, 2024

Conversation

michaelliao
Copy link
Owner

Reasons for making this change:

TODO

Links to documentation supporting these rule changes:

TODO

If this is a new template:

  • Link to application or project’s homepage: TODO

loozhengyuan and others added 6 commits December 27, 2023 22:59
`Directory.Build.rsp` is a [documented file][1] that allows setting default arguments to command line builds. However, our .gitignore template ignores _all_ `*.rsp` files. which causes confusion:

1. Devs write an .rsp file and if they aren't being attentive forget to commit it
2. Adding it to git requires `git add --force`, which some devs mistake for a destructive or not-recommended action

Thus, explicitly allow the `Directory.Build.rsp` file.

[1]: https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-response-files?view=vs-2022#directorybuildrsp
Move language is used for MoveVM based blockchains as the language
native to them.  The compiler provides a folder build/ that holds
the build artifacts.

The .aptos/ folder is ignored so people do not upload their private
keys to a git repository.
Exclude Directory.Build.rsp from VisualStudio.gitignore template
@michaelliao michaelliao merged commit 32244c4 into michaelliao:main Oct 18, 2024
1 check passed
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.

6 participants