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

fix(deps): bump to ember-get-config@1.0.0 #221

Merged

Conversation

rahulk94
Copy link
Contributor

Description

Bump to ember-get-config@1.0.0 so its versioning can be managed better in consumers. Fwiw we previously allowed versions up till 0.5.0 so this only really will be bumping the minimum version required up one (which was to add better Embroider support).

0.x versions are always a pain to deal with downstream so better to use 1.x now that its out.

Steps to Test

Link to a demo app and verify ember-cli-imgix still works as intended.

…r in consumers

0.x versions are always a pain to deal with downstream so better to use 1.x now that its out
@commit-lint
Copy link

commit-lint bot commented Jan 11, 2022

Contributors

rahulk94

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

@rahulk94
Copy link
Contributor Author

@frederickfogerty I've raised this as a draft for now since ember-get-config@1 was just released this morning. I'll mark it as ready for review in a few days incase they find some bugs that need hotfixing on their side before we go ahead and update this.

@rahulk94 rahulk94 marked this pull request as ready for review January 13, 2022 22:32
@rahulk94 rahulk94 requested a review from a team as a code owner January 13, 2022 22:32
Copy link
Contributor

@frederickfogerty frederickfogerty left a comment

Choose a reason for hiding this comment

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

LGTM

@frederickfogerty frederickfogerty changed the title chore(deps): bump to ember-get-config@1.0.0 fix(deps): bump to ember-get-config@1.0.0 Jan 20, 2022
@frederickfogerty frederickfogerty merged commit bd1a5a3 into imgix:main Jan 20, 2022
@rahulk94 rahulk94 deleted the deps/bump-to-ember-get-config@1 branch January 20, 2022 20:45
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.

2 participants