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

Remove Meteor.gitignore #1521

Merged
merged 1 commit into from Jun 11, 2015
Merged

Remove Meteor.gitignore #1521

merged 1 commit into from Jun 11, 2015

Conversation

lezed1
Copy link
Contributor

@lezed1 lezed1 commented May 14, 2015

There are only two entries in this .gitignore, and both are not needed.

  • .meteor/local is already ignored by the standard Meteor .gitignore that is included with every project. This is redundant and should b left solely to the standard Meteor .gitignore in case this changes in the future.
  • .meteor/meteorite should no longer be in any Meteor project. Meteorite was a package manager for Meteor, but it has been replaced by an official package manager built into the Meteor CLI. This directory should not exist, except on out of date Meteor projects which already have this ignore present.

There are only two entries in this .gitignore, and both are not needed.
- .meteor/local is already ignored by the standard Meteor .gitignore that is included with every project. This is redundant and should b left solely to the standard Meteor .gitignore in case this changes in the future.
- .meteor/meteorite should no longer be in any Meteor project. Meteorite was a package manager for Meteor, but it has been replaced by an official package manager built into the Meteor CLI. This directory should not exist, except on out of date Meteor projects which already have this ignore present.
@shiftkey
Copy link
Member

@lezed1 thanks for the details - do you have a link to some supporting docs about this so I can close this out?

@lezed1
Copy link
Contributor Author

lezed1 commented May 14, 2015

Here is the skeleton .gitignore with local already included: https://github.com/meteor/meteor/blob/devel/tools/skel/.meteor/.gitignore

And here is the web interface of the package manager demonstration the install command which no longer uses meteorite: https://atmospherejs.com/meteor/accounts-github

@lezed1
Copy link
Contributor Author

lezed1 commented Jun 11, 2015

Is there anything else needed to be done for this pull request?

@arcresu
Copy link
Contributor

arcresu commented Jun 11, 2015

Sorry for the wait; this looks good. Thanks! 🎉

arcresu added a commit that referenced this pull request Jun 11, 2015
@arcresu arcresu merged commit b304edf into github:master Jun 11, 2015
@lezed1 lezed1 deleted the patch-1 branch June 11, 2015 07:37
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants