-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add an alembic revision adding to CI notifs to the packager's defaut pack #225
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #225 +/- ##
========================================
Coverage 44.42% 44.42%
========================================
Files 70 70
Lines 3881 3881
Branches 496 496
========================================
Hits 1724 1724
Misses 2098 2098
Partials 59 59 Continue to review full report at Codecov.
|
continue | ||
|
||
modified += 1 | ||
filt.add_rule(session, valid_paths, path, negated=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure negated=True
is correct? Is the purpose of this migration to ignore all messages matched by ci_step_complete
that have to do with a user's packages?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To ignore the ones that have successfully completed, ie: notify about the ones that did not (that failed)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, okay.
target = "Events on packages that I own" | ||
|
||
|
||
def upgrade(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a comment detailing the purpose of the migration
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure
425456d
to
d73ee2a
Compare
… pack This setting will notify packager of any step of the CI pipeline that did not complete successfully. Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
d73ee2a
to
3abc010
Compare
This setting will notify packager of any step of the CI pipeline that
did not complete successfully.
Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr