Skip to content

Add git ignore for .dart_tool #1624

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

Merged
merged 3 commits into from
Mar 2, 2018
Merged

Add git ignore for .dart_tool #1624

merged 3 commits into from
Mar 2, 2018

Conversation

jcollins-g
Copy link
Contributor

Dart seems to generate a directory .dart_tool at the top of the project with snapshot files whenever pub runs now. Ignore files generated here.

@googlebot googlebot added the cla: yes Google CLA check succeeded. label Mar 2, 2018
@jcollins-g jcollins-g requested a review from kevmoo March 2, 2018 21:30
Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

This would be a great time to clean up the other noise, too. This should be all you need RE https://www.dartlang.org/guides/libraries/private-files

.dart_tool/
.packages
.pub/

But not critical...

@jcollins-g
Copy link
Contributor Author

Sorted the .gitignore and added a couple more minor entries for intellij.

@kevmoo
Copy link
Member

kevmoo commented Mar 2, 2018

Sorted the .gitignore and added a couple more minor entries for intellij.

Why not remove all of those and use .gitignore_global? https://help.github.com/articles/ignoring-files/#create-a-global-gitignore

@jcollins-g
Copy link
Contributor Author

jcollins-g commented Mar 2, 2018

That would work, though I generally prefer to minimize the amount of environment customization required to run or work on my tools. A global gitignore instead of one committed with the repository heads in the other way.

@jcollins-g jcollins-g merged commit 3c81fa3 into master Mar 2, 2018
@jcollins-g jcollins-g deleted the gitignore-dart_tool branch March 2, 2018 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants