Skip to content

chore: Bump gix-components #7101

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

chore: Bump gix-components #7101

wants to merge 14 commits into from
+115 −108

Conversation

AntonioVentilii
Copy link
Contributor

Motivation

We want to pull in the latest changes of GIX-cmp

Changes

  • Ran npm run update:gix
  • Update Component type.

Tests

  • CI should pass
  • The pulled in changes should have been tested before being committed to their repositories.

@AntonioVentilii AntonioVentilii requested a review from a team as a code owner July 10, 2025 12:27
@AntonioVentilii AntonioVentilii mentioned this pull request Jul 10, 2025
Closed
@AntonioVentilii
Copy link
Contributor Author

@mstrasinskis @yhabib how can I run the E2E tests? I cannot find them in the CI actions

@yhabib
Copy link
Contributor

yhabib commented Jul 10, 2025

They have already run, and they are green. 🚀
Screenshot 2025-07-10 at 23 26 55

@AntonioVentilii
Copy link
Contributor Author

Perfect, tks! May I please ask a review then?

@AntonioVentilii
Copy link
Contributor Author

AntonioVentilii commented Jul 11, 2025 •
edited
Loading

Fixing tests

Done!

@AntonioVentilii AntonioVentilii marked this pull request as draft July 11, 2025 13:20
@AntonioVentilii AntonioVentilii marked this pull request as ready for review July 11, 2025 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yhabib yhabib Awaiting requested review from yhabib

@peterpeterparker peterpeterparker Awaiting requested review from peterpeterparker

@mstrasinskis mstrasinskis Awaiting requested review from mstrasinskis

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AntonioVentilii @yhabib @peterpeterparker