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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore/release #38

Merged
merged 1 commit into from Sep 14, 2020
Merged

Conversation

RobertMaciejewski
Copy link
Contributor

@RobertMaciejewski RobertMaciejewski commented Sep 14, 2020

@iusztin
Prepared the release of the new version.
Link to draft release
https://github.com/NEWROPE/scnnr-ruby/releases/tag/untagged-7d3931b6d1959ddcbed0
Major update caused by #36 (review)
I want to confirm changelog and version choice - if it looks good I'll release it and after that rerun CI (I think it fails because of version change).

Please check 馃檹

Copy link
Member

@iusztin iusztin left a comment

Choose a reason for hiding this comment

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

Thank you for preparing the release :)
I agree about the new version & the changelog.

But about the failing RSpec I discovered something unfortunate 馃槗
The actual error is:


WebMock::NetConnectNotAllowedError:
  Real HTTP connections are disabled. Unregistered request: ...

After removing Gemfile.lock, running bunde and rspec I could reproduce it on my local.

I think it means Webmock fails to match some request that we make to a mocked request.

When checking the Gemfile I noticed we use a very wide version constrait for Webmock: https://guides.rubygems.org/patterns/#pessimistic-version-constraint and suspected it could have failed because of some breaking minor version change: https://github.com/bblimke/webmock/blob/master/CHANGELOG.md

Then I tried fixing webmock to a more precise version like gem 'webmock', '~> 3.0.0' and installed the bundle and ran rspec again -> passed on my local.

-> Sorry, could you take a closer look at this again? (and choose an appropriate Webmock version if needed :))

@RobertMaciejewski
Copy link
Contributor Author

RobertMaciejewski commented Sep 14, 2020

@iusztin
I couldn't reproduce it locally 馃槃
Turns out the version that was not working was 3.9.0 (which you were installing locally), but they managed to fix it in the meantime (about 7h ago) in 3.9.1. So the issue no longer occurs. After restarting CI build it works!
https://github.com/bblimke/webmock/blob/master/CHANGELOG.md#391

@iusztin
Copy link
Member

iusztin commented Sep 14, 2020

Ah, super unlucky - or lucky? lol.
Great find :) Thanks!

Copy link
Member

@iusztin iusztin left a comment

Choose a reason for hiding this comment

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

Thank you, LGTM!

@iusztin iusztin merged commit f965088 into NEWROPE:master Sep 14, 2020
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

2 participants