Skip to content

Conversation

@saaivignesh20
Copy link
Collaborator

No description provided.

@saaivignesh20 saaivignesh20 marked this pull request as ready for review October 9, 2025 23:52
@saaivignesh20 saaivignesh20 marked this pull request as draft October 9, 2025 23:52
@nx-cloud
Copy link

nx-cloud bot commented Oct 9, 2025

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 194fbfe

Command Status Duration Result
nx affected --target=build,build-webcomponent,b... ❌ Failed 2m 44s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-03 21:16:57 UTC

@github-actions
Copy link

github-actions bot commented Oct 10, 2025

🚀 Preview Deploy Report

✅ Successfully deployed preview here

@axdanbol axdanbol linked an issue Nov 20, 2025 that may be closed by this pull request
@axdanbol axdanbol changed the base branch from populate-test-coverage-new to develop November 21, 2025 15:40
@axdanbol
Copy link
Contributor

axdanbol commented Nov 21, 2025

@saaivignesh20 It might not be feasible to continue work on this branch due to all the dummy test files and other out of date changes. Instead I suggest you use git checkout origin/test-coverage-asctb-reporter -- apps/asctb-reporter/* to cherry pick changes to a new branch and create a new PR. Also you don't need to get full coverage with a single PR, it is perfectly fine to create many smaller PRs that increase the coverage for only a part of the asctb-reporter each

@saaivignesh20
Copy link
Collaborator Author

Closing this pull request, raising a simpler one as per @axdanbol's suggestion.

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.

asctb-reporter: Increase test coverage

3 participants