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 the patternfly group with 2 updates #21399

Merged
merged 2 commits into from
Dec 10, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 8, 2024

Bumps the patternfly group with 2 updates: @patternfly/react-core and @patternfly/react-table.

Updates @patternfly/react-core from 5.4.10 to 5.4.11

Commits
  • 2c6037e chore(release): releasing packages [ci skip]
  • 623da5e feat(V5 Select/Dropdown/MenuContainer): arrow key handling to focus items (#1...
  • af540d5 chore(deps): update dependency sass to ^1.81.0 (#11258)
  • ceb5d0e chore(deps): update dependency rollup to ^4.27.3 (#11257)
  • a7657ef chore(deps): update dependency globals to ^15.12.0 (#11255)
  • d0af1c8 chore(deps): update dependency eslint-plugin-testing-library to ^6.5.0 (#11254)
  • 75638d7 chore(deps): update dependency @​testing-library/jest-dom to ^6.6.3 (#11251)
  • See full diff in compare view

Updates @patternfly/react-table from 5.4.11 to 5.4.12

Commits
  • 2c6037e chore(release): releasing packages [ci skip]
  • 623da5e feat(V5 Select/Dropdown/MenuContainer): arrow key handling to focus items (#1...
  • af540d5 chore(deps): update dependency sass to ^1.81.0 (#11258)
  • ceb5d0e chore(deps): update dependency rollup to ^4.27.3 (#11257)
  • a7657ef chore(deps): update dependency globals to ^15.12.0 (#11255)
  • d0af1c8 chore(deps): update dependency eslint-plugin-testing-library to ^6.5.0 (#11254)
  • 75638d7 chore(deps): update dependency @​testing-library/jest-dom to ^6.6.3 (#11251)
  • See full diff in compare view

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 <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

@github-actions github-actions bot changed the title [no-test]: Bump the patternfly group with 2 updates Bump the patternfly group with 2 updates Dec 8, 2024
@allisonkarlitskaya allisonkarlitskaya force-pushed the dependabot/npm_and_yarn/patternfly-65db92bc16 branch 2 times, most recently from 66874c0 to bacd668 Compare December 8, 2024 04:11
@martinpitt
Copy link
Member

@mvollmer This is a PF change to the TypeaheadSelect component. Is this legit or a regression? Can you please have a look? Thanks!

@mvollmer
Copy link
Member

mvollmer commented Dec 9, 2024

The test was always flaky, because of asynchronicity and general messedupnes of the PF typeahed template. (I am getting more and more disillusioned with PF code quality.)

@mvollmer
Copy link
Member

mvollmer commented Dec 9, 2024

The test was always flaky, because of asynchronicity and general messedupnes of the PF typeahed template. (I am getting more and more disillusioned with PF code quality.)

That was unfair: the messedupnes only exists with this new version of PF. All code based on the templates and on the old examples needs to be updated.

@allisonkarlitskaya allisonkarlitskaya force-pushed the dependabot/npm_and_yarn/patternfly-65db92bc16 branch from aeafea9 to c66ab56 Compare December 9, 2024 14:07
@martinpitt
Copy link
Member

martinpitt commented Dec 9, 2024

Nice, thanks @mvollmer ! This mypy error is fixed in #21410. But how can this happen? The unit test already uses the pinned .cockpit-ci/container tag, not :latest. 🤯 Update: #21387 and #21385 collided.

Also, node_modules is somehow out of date.

I'll deal with this after #21410 lands.

mvollmer and others added 2 commits December 10, 2024 08:23
… changes

The new PF version will move the keyboard focus into the SelectList of
a opened Select when the user presses ArrowUp or ArrowDown
keys. Further keyboard navigation then happens by moving the focus
around in that list.

However, TypeaheadSelect and MultiTypeaheadSelect templated implement
their own custom keyboard navigation based on the (now broken)
assumption that the text input stays in focus all the time.

To restore the assumption, Select has a new "typeahead" variant that
switches off its new key handling, letting the Typeahead templates
live in the past for a little while longer.

See patternfly/patternfly-react#11249
Bumps the patternfly group with 2 updates: [@patternfly/react-core](https://github.com/patternfly/patternfly-react) and [@patternfly/react-table](https://github.com/patternfly/patternfly-react).

Updates `@patternfly/react-core` from 5.4.10 to 5.4.11
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

Updates `@patternfly/react-table` from 5.4.11 to 5.4.12
- [Release notes](https://github.com/patternfly/patternfly-react/releases)
- [Commits](https://github.com/patternfly/patternfly-react/compare/@patternfly/[email protected]...@patternfly/[email protected])

---
updated-dependencies:
- dependency-name: "@patternfly/react-core"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: patternfly
- dependency-name: "@patternfly/react-table"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: patternfly
...

Signed-off-by: dependabot[bot] <[email protected]>
@martinpitt martinpitt force-pushed the dependabot/npm_and_yarn/patternfly-65db92bc16 branch from c66ab56 to c1432b9 Compare December 10, 2024 07:24
Copy link
Member

@martinpitt martinpitt left a comment

Choose a reason for hiding this comment

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

Fixed up. This should go green now.

@martinpitt martinpitt merged commit 8cd5c50 into main Dec 10, 2024
84 of 85 checks passed
@martinpitt martinpitt deleted the dependabot/npm_and_yarn/patternfly-65db92bc16 branch December 10, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants