Skip to content

Conversation

@Mariatta
Copy link
Member

@Mariatta Mariatta commented Sep 4, 2020

Look up conclusions from check runs in addition to status checks.

Ensure all check runs have completed.
Ensure there is no "failure" or "timed_out" result from the check runs

Fixes #375

Look up conclusions from check runs in addition to status checks.

Ensure all check runs have completed.
Ensure there is no "failure" or "timed_out" result from the check runs

Fixes #375
@Mariatta Mariatta added black out Apply to PR. It will run "black" and removed CLA signed labels Sep 4, 2020
@black-out
Copy link

black-out bot commented Sep 4, 2020

🐍🌚🤖 @Mariatta, I've formatted these files using black:

  • tests/test_status_change.py
    (I'm a bot 🤖)

@black-out black-out bot removed the black out Apply to PR. It will run "black" label Sep 4, 2020
@codecov
Copy link

codecov bot commented Sep 4, 2020

Codecov Report

Merging #382 into main will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
+ Coverage   96.42%   96.51%   +0.09%     
==========================================
  Files           9        9              
  Lines         924      948      +24     
  Branches       55       58       +3     
==========================================
+ Hits          891      915      +24     
  Misses         33       33              
Impacted Files Coverage Δ
miss_islington/status_change.py 100.00% <100.00%> (ø)
miss_islington/util.py 100.00% <100.00%> (ø)
tests/test_status_change.py 100.00% <100.00%> (ø)

@Mariatta Mariatta temporarily deployed to miss-islington September 4, 2020 20:56 Inactive
@Mariatta Mariatta merged commit dc67f1e into main Sep 4, 2020
@Mariatta Mariatta deleted the fix-issue-375 branch September 4, 2020 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automerge is broken after travis ci migration

3 participants