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

[FINE] Add previous migrations to .gitignore #15321

Merged
merged 1 commit into from
Jun 12, 2017
Merged

[FINE] Add previous migrations to .gitignore #15321

merged 1 commit into from
Jun 12, 2017

Conversation

djberg96
Copy link
Contributor

@djberg96 djberg96 commented Jun 6, 2017

Just a minor mod to the .gitignore file for the Fine branch so that I don't see spec/replication/util/data/previous_migrations as untracked files.

.gitignore Outdated
@@ -73,6 +73,7 @@ public/ui/

# spec/
spec/replication/util/data/euwe_migrations
spec/replication/util/data/previous_migrations
Copy link
Member

Choose a reason for hiding this comment

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

master has *_migrations...can this PR just do the same to keep it consistent?

https://github.com/ManageIQ/manageiq/blob/master/.gitignore#L76-L77

@Fryguy Fryguy changed the title [Fine] Add previous migrations to .gitignore [FINE] Add previous migrations to .gitignore Jun 6, 2017
@djberg96
Copy link
Contributor Author

djberg96 commented Jun 6, 2017

@Fryguy Done.

@miq-bot
Copy link
Member

miq-bot commented Jun 6, 2017

Checked commit https://github.com/djberg96/manageiq/commit/ed8e7376ade45c7179ebe72e1f7a15d027e5ba81 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@chrisarcand chrisarcand merged commit ca3d55a into ManageIQ:fine Jun 12, 2017
@chrisarcand chrisarcand added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants