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

ci(custom-checks): fix dependencies being inconsistent phase 1 #3684

Open
jagpreetsinghsasan opened this issue Dec 10, 2024 · 2 comments
Open
Assignees
Labels
dependent enhancement New feature or request P3 Priority 3: Medium

Comments

@jagpreetsinghsasan
Copy link
Contributor

jagpreetsinghsasan commented Dec 10, 2024

Description

As a developer, I want to fix the dependency versions being inconsistent across packages in Cacti.

Targeted packages:

packages/cacti-ledger-browser
packages/cacti-plugin-consortium-static (This one will be tough imo as we need to upgrade web3 from 1.x -> 4.x)

Depends on #3612

Solution:

Run the script tools/custom-checks/check-dependency-version-consistency.ts with fix: true, note down the changes for the above targeted packages, stash and drop the changes, and proceed with fixing just those targeted packages.

Possible errors

  1. The package being dependent on other packages can lead to conflicts in those dependent packages. Even if those newly failing package tests occur, fixing those in this phase becomes important or else we will have broken code in repo.
    To test this scenario, simply create a PR for this task fixing only the targeted packages and have a lookout on CI for any failing tests. If we have any failing tasks, we should fix them as a part of this task
  2. The fix might involve breaking changes, and shall be reported in the PR
  3. For some dependencies, the suggest change might not be an exact version (due to it being the highest version among all packages), so instead of including the versions like >=1.2.3, fix the root package which has this sort of version and then include that in the conflicting package (a 2 way solution)

Acceptance Criteria

  1. The above targeted packages have all the dependency version inconsistencies fixed
  2. No new crashes are seen in CI
@jagpreetsinghsasan jagpreetsinghsasan added the enhancement New feature or request label Dec 10, 2024
Copy link

This PR/issue depends on:

@petermetz
Copy link
Contributor

@jagpreetsinghsasan Agreed, anything that touches on the web3 deps will be quite difficult in my opinion as well. Let's focus on the low hanging fruit first!

@petermetz petermetz added the P3 Priority 3: Medium label Dec 13, 2024
@jagpreetsinghsasan jagpreetsinghsasan self-assigned this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependent enhancement New feature or request P3 Priority 3: Medium
Projects
None yet
Development

No branches or pull requests

2 participants