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.

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

chore: package dependencies #1299

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

chore: package dependencies #1299

wants to merge 2 commits into from

Conversation

gagdiez
Copy link
Collaborator

@gagdiez gagdiez commented Feb 11, 2025

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Type of change.

  • FIX - a PR of this type patches a bug.
  • FEATURE - a PR of this type introduces a new feature.
  • BUILD - a PR of this type introduces build changes.
  • CI - a PR of this type introduces CI changes.
  • DOCS - a PR of this type introduces DOCS improvement.
  • STYLE - a PR of this type introduces style changes.
  • REFACTOR - a PR of this type introduces refactoring.
  • PERFORMANCE - a PR of this type introduces performance changes.
  • TEST - a PR of this type adds more tests.
  • CHORE - a PR introduces other changes than the specified above.

@gagdiez
Copy link
Collaborator Author

gagdiez commented Feb 11, 2025

We forgot to update the version of wallet-selector react-hook on release

@gagdiez gagdiez changed the title fix: package dependencies chore: package dependencies Feb 11, 2025
trechriron
trechriron previously approved these changes Feb 11, 2025
View reviewed changes
Copy link
Collaborator

@trechriron trechriron left a comment

Choose a reason for hiding this comment

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

LGTM

@trechriron trechriron changed the base branch from main to dev February 11, 2025 15:54
@trechriron trechriron dismissed their stale review February 11, 2025 15:54

The base branch was changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flmel flmel flmel approved these changes

@trechriron trechriron trechriron left review comments

At least 2 approving reviews are 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
@gagdiez @trechriron @flmel