Skip to content

Resolve pinned dependencies#1551

Open
Akshay2191 wants to merge 4 commits intomainfrom
resolve-pinned-dependencies
Open

Resolve pinned dependencies#1551
Akshay2191 wants to merge 4 commits intomainfrom
resolve-pinned-dependencies

Conversation

@Akshay2191
Copy link
Copy Markdown
Contributor

Proposed changes

I have fixed all the Pinned Dependencies which are doable, the remaining ones are in Test Docker file.
These test Dockerfiles use dynamic base images (${BASE_IMAGE}) by design and cannot be pinned without breaking test flexibility.

I have also added a change in the workflow so that we can run the dependencies test on any branch(for testing purpose).

@Akshay2191 Akshay2191 requested a review from a team as a code owner March 4, 2026 13:29
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.02%. Comparing base (2936689) to head (8f2ab52).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1551      +/-   ##
==========================================
+ Coverage   83.52%   85.02%   +1.50%     
==========================================
  Files         103      103              
  Lines       16250    13589    -2661     
==========================================
- Hits        13572    11554    -2018     
+ Misses       2161     1518     -643     
  Partials      517      517              

see 100 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2936689...8f2ab52. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants