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

faraday: allow latest versions #144

Closed
wants to merge 2 commits into from

Conversation

bastelfreak
Copy link

latest faraday is required because:
beaker -> beaker-abs -> vmfloaty

and at least in beaker we need the github changelog generator gem and
that requires faraday 1.x

Status

[Ready for Merge | In Progress | ???]

Description

FIXME

Related Issues

Todos

  • Tests
  • Documentation

Reviewers

@puppetlabs/dio
@highb
@briancain

latest faraday is required because:
beaker -> beaker-abs -> vmfloaty

and at least in beaker we need the github changelog generator gem and
that requires faraday 1.x
@bastelfreak bastelfreak requested review from briancain, highb and a team as code owners July 11, 2021 18:52
bastelfreak added a commit to bastelfreak/beaker-abs that referenced this pull request Jul 11, 2021
This is required to pull in the latest vmfloaty release with in turn
allow the latest faraday (hopefully). See puppetlabs/vmfloaty#144
for details.
this is required to allow latest faraday
Copy link
Contributor

@genebean genebean left a comment

Choose a reason for hiding this comment

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

Please adjust this to some kind of bounded version instead of the new open ended ones shown here.

@genebean
Copy link
Contributor

@bastelfreak I am fairly sure that updating faraday and webmock is going to require some updating to tests too as I think there are breaking changes in newer versions.

@highb
Copy link
Contributor

highb commented Jul 13, 2021

If I'm grokking the versions correctly, #145 has handled this. Closing.

@highb highb closed this Jul 13, 2021
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.

3 participants