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

Release version 4.1.0. #92

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Release version 4.1.0. #92

merged 3 commits into from
Oct 3, 2022

Conversation

sengi
Copy link
Contributor

@sengi sengi commented Oct 3, 2022

If we wanted to be super strict, we could view removing the unused setter methods for parent_domain and external_domain as a compatibility-breaking API change. There isn't a single usage of these anywhere though, so in practice it's more helpful to consider this a minor release as it's supposed to be 100% compatible with all existing usage.

Also, start using https when fetching gem dependencies during build.

This has been the default for new Gemfiles since a major security
incident at rubygems.org in 2013[1].

While we're at it, also fix a missing https in a link in the README.

[1]: https://venturebeat.com/business/rubygems-org-hacked-interrupting-heroku-services-and-putting-millions-of-sites-using-rails-at-risk/
@sengi sengi requested a review from kevindew October 3, 2022 11:30
@sengi sengi changed the title Release version 5.0.0. Release version 4.1.0. Oct 3, 2022
@sengi
Copy link
Contributor Author

sengi commented Oct 3, 2022

On second thoughts, let's release a 4.1.0 now to get us unblocked and then include some actually-breaking stuff in 5.0 shortly after. That way we don't risk blocking Replatforming on changes that aren't strictly needed for Replatforming, but we can still move just as fast on the other Plek cleanup stuff in the likely event that it goes smoothly.

kevindew and others added 2 commits October 3, 2022 14:45
This method was put in temporarily 10 years ago [1] and has had a todo
next to it ever since. We're planning to finally remove this [2] in the
5.0 release so this acts as a warning in the interim.

[1]: 309ef59
[2]: #93
Copy link
Member

@kevindew kevindew left a comment

Choose a reason for hiding this comment

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

Looks good, I did a quick rebase to get commits in order of release

@sengi sengi merged commit 0f9c8e8 into main Oct 3, 2022
@sengi sengi deleted the sengi/version branch October 3, 2022 14:02
kevindew added a commit to alphagov/gds-api-adapters that referenced this pull request Oct 3, 2022
This has been deprecated [1] since Plek 4.1 [2]. Instead we can use
forego the need for a current method and call find directly.

[1]: alphagov/plek#94
[2]: alphagov/plek#92
kevindew added a commit to alphagov/gds-api-adapters that referenced this pull request Oct 3, 2022
This has been deprecated [1] since Plek 4.1 [2]. Instead we can use
forego the need for a current method and call find directly.

[1]: alphagov/plek#94
[2]: alphagov/plek#92
kevindew added a commit to alphagov/govuk_publishing_components that referenced this pull request Oct 3, 2022
Plek 4.1 [1] deprecates the Plek.current method. This resolves this gem
emitting deprecation warnings.

[1]: alphagov/plek#92
kevindew added a commit to alphagov/govuk_publishing_components that referenced this pull request Oct 3, 2022
Plek 4.1 [1] deprecates the Plek.current method. This resolves this gem
emitting deprecation warnings.

This also updates the Gemfile.lock of this gem to use the new version.

[1]: alphagov/plek#92
kevindew added a commit to alphagov/govuk_publishing_components that referenced this pull request Oct 3, 2022
Plek 4.1 [1] deprecates the Plek.current method. This resolves this gem
emitting deprecation warnings.

This also updates the Gemfile.lock of this gem to use the new version.

[1]: alphagov/plek#92
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