Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci(custom-checks): fix dependencies being inconsistent phase 2 #3685

Open
jagpreetsinghsasan opened this issue Dec 10, 2024 · 1 comment
Open
Labels
dependent enhancement New feature or request

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-plugin-copm-fabric 
packages/cacti-plugin-ledger-connector-stellar
packages/cactus-api-client
packages/cactus-cmd-api-server
packages/cactus-common

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:

@jagpreetsinghsasan jagpreetsinghsasan added Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) and removed Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) labels Dec 10, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
dependent enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant