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

[ALOY-1609] Add plugins/ti.alloy to gitignore #883

Merged
merged 2 commits into from Mar 7, 2018
Merged

[ALOY-1609] Add plugins/ti.alloy to gitignore #883

merged 2 commits into from Mar 7, 2018

Conversation

Topener
Copy link
Sponsor Contributor

@Topener Topener commented Mar 7, 2018

JIRA: https://jira.appcelerator.org/browse/ALOY-1609

Every time the alloy version changes, the alloy.js plugin file inside the plugin folder changes too. This pollutes git and a default procedure is to remove the file from git alltogether.

Copy link
Contributor

@hansemannn hansemannn left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! @feons are you okay with this change?

@hansemannn hansemannn requested a review from feons March 7, 2018 14:29
@@ -10,3 +10,4 @@ tmp
.project
.settings
Thumbs.db
plugins/ti.alloy/hooks/alloy.js
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 we'd want to ignore plugins/ti.alloy and not the specific file? There's a couple other files under there that can change (hooks/readme, hooks/deepclean.js, plugin.py)

Copy link
Sponsor Contributor Author

Choose a reason for hiding this comment

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

yep that actually makes more sense! Will update the PR

@hansemannn hansemannn changed the title [ALOY-1609] Add plugins/ti.alloy/hooks/alloy.js to gitignore [ALOY-1609] Add plugins/ti.alloy to gitignore Mar 7, 2018
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.

None yet

4 participants