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

#31 Bump dependencies for packages/ui #36

Merged
merged 8 commits into from
Jul 24, 2023
Merged

#31 Bump dependencies for packages/ui #36

merged 8 commits into from
Jul 24, 2023

Conversation

nevendyulgerov
Copy link
Contributor

Base branch for all changes related to #31 . Sub-branches will tackle most of the work for it and I'll merge those back once they are approved.

@nevendyulgerov nevendyulgerov added type:chore It is not a new feature or bug fix. e.g. update dependencies, change repository configurations package:ui changes targeting shared UI package labels Jul 6, 2023
@nevendyulgerov nevendyulgerov self-assigned this Jul 6, 2023
@vercel
Copy link

vercel bot commented Jul 6, 2023 •
edited

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rollups ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 24, 2023 8:21am
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 24, 2023 8:21am

@nevendyulgerov nevendyulgerov linked an issue Jul 6, 2023 that may be closed by this pull request
Bump dependencies for UI package #31
Closed
8 tasks
@github-actions
Copy link

github-actions bot commented Jul 6, 2023 •
edited

Pull Request Test Coverage Report for Build 5599039768

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 20 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.09%) to 73.047%
Files with Coverage Reduction New Missed Lines %
apps/staking/src/components/CTSIText.tsx 9 64.58%
apps/staking/src/components/node/OwnedNode.tsx 11 0%
Totals Coverage Status
Change from base Build 5576343280: -0.09%
Covered Lines: 2560
Relevant Lines: 3340
💛 - Coveralls

@nevendyulgerov nevendyulgerov force-pushed the chore/31-bump-dependencies-for-ui-package branch from 84d85b1 to b67bfe1 Compare July 19, 2023 12:23
@nevendyulgerov nevendyulgerov marked this pull request as ready for review July 19, 2023 13:32
@nevendyulgerov nevendyulgerov changed the title #31 Bump dependencies for UI package #31 Bump dependencies for packages/ui Jul 21, 2023
brunomenezes
brunomenezes requested changes Jul 23, 2023
View reviewed changes
Copy link
Collaborator

@brunomenezes brunomenezes left a comment

Choose a reason for hiding this comment

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

@nevendyulgerov,
I just merged your other PR for the packages/utils, There is a conflict to be solved in the yarn.lock.

@nevendyulgerov nevendyulgerov force-pushed the chore/31-bump-dependencies-for-ui-package branch from a5d9525 to 3c40e74 Compare July 24, 2023 08:11
@brunomenezes brunomenezes merged commit 3afc7bf into main Jul 24, 2023
8 of 9 checks passed
@brunomenezes brunomenezes deleted the chore/31-bump-dependencies-for-ui-package branch July 24, 2023 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brunomenezes brunomenezes brunomenezes approved these changes

@dandheedge dandheedge Awaiting requested review from dandheedge

Assignees

@nevendyulgerov nevendyulgerov

Labels
package:ui changes targeting shared UI package type:chore It is not a new feature or bug fix. e.g. update dependencies, change repository configurations
Projects
Explorer Unit
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bump dependencies for UI package
2 participants
@nevendyulgerov @brunomenezes