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

Bump govuk_publishing_components from 37.2.4 to 37.3.0 #3246

Merged
merged 2 commits into from
Feb 7, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 6, 2024

Bumps govuk_publishing_components from 37.2.4 to 37.3.0.

Changelog

Sourced from govuk_publishing_components's changelog.

37.3.0

Commits
  • 8e52c13 Merge pull request #3857 from alphagov/version-37.3.0
  • f1efcfd Lock govuk-frontend version to 4.7.0
  • ef7fa2f Version 37.3.0
  • 971b4b0 Merge pull request #3842 from alphagov/dependabot/bundler/rubocop-govuk-4.14.0
  • 095e0f2 Merge pull request #3853 from alphagov/dependabot/bundler/nokogiri-1.16.2
  • 834243f Merge pull request #3851 from alphagov/separate-env-code
  • 4bdd776 Update changelog
  • 702a54a Allow other analytics domain details to load
  • 580ac00 Bump nokogiri from 1.16.0 to 1.16.2
  • 5e02f61 Merge pull request #3830 from alphagov/hide-cookie-banner-js-disabled
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [govuk_publishing_components](https://github.com/alphagov/govuk_publishing_components) from 37.2.4 to 37.3.0.
- [Changelog](https://github.com/alphagov/govuk_publishing_components/blob/main/CHANGELOG.md)
- [Commits](alphagov/govuk_publishing_components@v37.2.4...v37.3.0)

---
updated-dependencies:
- dependency-name: govuk_publishing_components
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies ruby Pull requests that update Ruby code labels Feb 6, 2024
@MartinJJones
Copy link
Contributor

The reason for the failing tests is because the Cookie Banner is now hidden by default when JavaScript is disabled, see alphagov/govuk_publishing_components#3830

Removing the checks in the 2 files below will fix this:

As the cookie banner is now hidden when JS is disabled, these tests can't see the banner unless this code is added
@AshGDS AshGDS force-pushed the dependabot/bundler/govuk_publishing_components-37.3.0 branch from 86952da to 6b793ca Compare February 6, 2024 16:39
@AshGDS
Copy link
Contributor

AshGDS commented Feb 6, 2024

@MartinJJones Thanks - I managed to get them to pass - are you able to review my commit?

Copy link
Contributor

@MartinJJones MartinJJones left a comment

Choose a reason for hiding this comment

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

Nice fix @AshGDS , looks good to me 👍

@AshGDS AshGDS merged commit 27826bd into main Feb 7, 2024
12 checks passed
@AshGDS AshGDS deleted the dependabot/bundler/govuk_publishing_components-37.3.0 branch February 7, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants