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

Optionally symlink a plugin's spec/manageiq #19481

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Nov 7, 2019

Add the option to symlink a plugin's spec/manageiq instead of a shallow
clone, allowing for local repositories to be linked together to test
groups of changes.

@agrare agrare requested a review from Fryguy November 7, 2019 20:49
@agrare
Copy link
Member Author

agrare commented Nov 7, 2019

@Fryguy figured we could work out how the generator should look then I'll blast out the rest of the plugins from this

@agrare agrare force-pushed the optionally_symlink_spec_manageiq_in_plugins branch from 1b3bced to d330e4b Compare November 7, 2019 21:07
@agrare agrare force-pushed the optionally_symlink_spec_manageiq_in_plugins branch from d330e4b to ddd1323 Compare November 7, 2019 21:10
Add the option to symlink a plugin's spec/manageiq instead of a shallow
clone, allowing for local repositories to be linked together to test
groups of changes.
@miq-bot
Copy link
Member

miq-bot commented Nov 8, 2019

Checked commits agrare/manageiq@ddd1323~...e21c51f with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@miq-bot
Copy link
Member

miq-bot commented Nov 8, 2019

Some comments on commits agrare/manageiq@ddd1323~...e21c51f

lib/generators/manageiq/plugin/templates/bin/setup

  • ⚠️ - 10 - Detected puts. Remove all debugging statements.

@Fryguy Fryguy merged commit 2b0f2a6 into ManageIQ:master Nov 8, 2019
@Fryguy Fryguy added this to the Sprint 124 Ending Nov 11, 2019 milestone Nov 8, 2019
@agrare agrare deleted the optionally_symlink_spec_manageiq_in_plugins branch November 8, 2019 00:53
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.

3 participants