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

remove invalid holdings after audit #108

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

cjyetman
Copy link
Member

@cjyetman cjyetman commented Jan 26, 2025

Copy link

codecov bot commented Jan 26, 2025

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 17.92%. Comparing base (c923fb6) to head (f4ac069).

Files with missing lines Patch % Lines
R/run_audit.R 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   18.00%   17.92%   -0.09%     
==========================================
  Files           4        4              
  Lines         422      424       +2     
==========================================
  Hits           76       76              
- Misses        346      348       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

github-actions bot commented Jan 26, 2025

Docker build status

commit_time git_sha image
2025-01-27T13:58:06Z f4ac069 ghcr.io/rmi-pacta/workflow.pacta:pr-108

@cjyetman
Copy link
Member Author

Looks like the CIs that are failing are due to this unrelated issue Azure/azure-cli#30635

cjyetman added a commit to RMI-PACTA/workflow.transition.monitor that referenced this pull request Jan 28, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant