Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

ci: enable code coverage reporting on neutron-understack #253

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

cardoe
Copy link
Contributor

@cardoe cardoe commented Sep 4, 2024

Enable code coverage reporting on neutron-understack and de-dup the rules.

@cardoe cardoe requested a review from a team September 4, 2024 16:41
skrobul added a commit that referenced this pull request Sep 9, 2024
This reverts commits 952f8c3 and 27f5b2f.

These changes have already been implemented in:

#251
#253

Both of the PRs solve the same problem, so I don't need to solve it here
again. It was initially fixed only because I was not aware of the other
PRs. Reverting instead of dropping these commits, because they have been
referenced externally in the comments.
skrobul added a commit that referenced this pull request Sep 9, 2024
This reverts commits 3f0e9f8 and b4961f4

These changes have already been implemented in:

#251
#253

Both of the PRs solve the same problem, so I don't need to solve it here
again. It was initially fixed only because I was not aware of the other
PRs. Reverting instead of dropping these commits, because they have been
referenced externally in the comments.
skrobul added a commit that referenced this pull request Sep 9, 2024
This reverts commits 664312d8e1828e5608671b0b2853f8c62ba0e165 and a7819cf
These changes have already been implemented in:

#251
#253

Both of the PRs solve the same problem, so I don't need to solve it here
again. It was initially fixed only because I was not aware of the other
PRs. Reverting instead of dropping these commits, because they have been
referenced externally in the comments.
Trigger these jobs on any change in the python/ path. De-dupped the
rules to avoid bad copy and paste in the paths, specifically around the
coverage which had checked the ironic code coverage instead.
@cardoe cardoe changed the title ci: enable testing on neutron-understack ci: enable code coverage reporting on neutron-understack Sep 11, 2024
@cardoe cardoe merged commit 7805e8b into main Sep 11, 2024
17 checks passed
@cardoe cardoe deleted the neutron-pytest branch September 11, 2024 13:30
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