-
Notifications
You must be signed in to change notification settings - Fork 28
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 miq specific source #209
Conversation
We need to have only one source per Gemfile
Checked commit kbrock@265e07e with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint |
LGTM, but do you know why was it added in the first place? |
@Fryguy thanks for keeping me honest. I added the notes |
Thanks, I found it after you did most of the work. At the time, this repo had manageiq-gems-pending in the Gemfile: manageiq-gems-pending included handsoap until the vmware stuff was extracted: ManageIQ/manageiq-gems-pending#169 Also, we dropped the gems pending dependency from the gemfile in this repo here: #153 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for digging in and figuring it out
Remove miq specific source (cherry picked from commit bb61b83)
Fixed - Fix sporadic test failure [#204] - Remove MIQ specific gem source [#209] - Double escape @ in realm to avoid shell interpretation [#211] - Move gem name loader to proper namespaced location [#208] - Separate kerberos from service principal name and use correctly [#215] - Add manageiq user to allowed_uids for sssd [#220] - Remove warning about using pg_dump [#221] - Fix specs where AwesomeSpawn private interface changed [#224] - Change the Name of the CA from something to ApplianceCA [#228] - Fix YAML.load_file failing on aliases [#234] Added - Make backward compatible changes to work with repmgr13 - version 5.2.1 [#192] - Support Ruby 3.0 [#206] - Support Ruby 3.1 [#227] - Allow rails 7 gems in gemspec [#226] Changed - Update to Highline 2.1.0 [#201] - Clean up test output (highline and stdout messages) [#210] Removed - Drop Ruby 2.7 [#223]
Fixes broken CI failure
We need to have only one main source in a
Gemfile
handsoap
dependency in Add rubygems.manageiq.org for handsoap dependency #109 to fix a build failure.Gemfile.lock
does not mention any manageiq dependencies (besidesmanageiq-password
) so I don't see a way we could inadvertently requirehandsoap
.