Skip to content

Conversation

@mineme0110
Copy link
Contributor

@mineme0110 mineme0110 commented Dec 9, 2025

Description

Summarize the changes you're submitting in a few sentences, including Jira ticket ATL-xxxx if applicable.
Link to any discussion, related issues and bug reports to give the context to help the reviewer understand the PR.

Alternatives Considered (optional)

Link to existing ADR (Architecture Decision Record), if any. If relevant, describe other approaches explored and the selected approach. Documenting why the methods were not selected will create a knowledge base for future reference, helping prevent others from revisiting less optimal ideas.

Checklist

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
@mineme0110 mineme0110 requested a review from a team as a code owner December 9, 2025 23:57
@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

Integration Test Results

 24 files  ±0   24 suites  ±0   3s ⏱️ ±0s
 61 tests ±0   61 ✅ ±0  0 💤 ±0  0 ❌ ±0 
113 runs  ±0  113 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit adbdb0e. ± Comparison against base commit 6d6a65e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

Unit Test Results

106 files  ±0  106 suites  ±0   16m 57s ⏱️ - 3m 22s
899 tests ±0  891 ✅ ±0  8 💤 ±0  0 ❌ ±0 
906 runs  ±0  898 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit adbdb0e. ± Comparison against base commit 6d6a65e.

♻️ This comment has been updated with latest results.

@coveralls
Copy link

coveralls commented Dec 10, 2025

Coverage Status

coverage: 47.662%. remained the same
when pulling adbdb0e on pinned-dependencies
into 6d6a65e on main.

Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
@mineme0110 mineme0110 changed the title fix: WIP Pinned dependencies fix: pinned dependencies Dec 10, 2025
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
run: |
./docker/run.sh "$AGENT_VERSION"
npm i
npm install
Copy link
Contributor

Choose a reason for hiding this comment

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

for pinned version we should use npm ci

Copy link
Contributor Author

@mineme0110 mineme0110 Dec 11, 2025

Choose a reason for hiding this comment

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

I will see what can be done it is bit tricky since no package-lock.json

Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
@sonarqubecloud
Copy link

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.

4 participants