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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automerge PR labeled with "automerge". #146

Merged
merged 6 commits into from Sep 11, 2018

Conversation

Projects
None yet
4 participants
@Mariatta
Member

Mariatta commented Sep 10, 2018

Merge the PR if:

  • PR has "awaiting merge" and "馃 automerge" labels, and
  • all CI status checks have passed, and
  • replace # with GH-
  • get the commit message from PR title and description

Before deploying:

  • Create "馃 automerge" label in CPython.

Closes python/bedevere#14

Closes python/core-workflow#29

@miss-islington

This comment has been minimized.

Collaborator

miss-islington commented Sep 10, 2018

馃悕馃寶馃 @Mariatta, I've formatted these files using black:

  • miss_islington/status_change.py
  • miss_islington/util.py
  • tests/test_status_change.py
  • tests/test_util.py
    (I'm a bot 馃)
@codecov

This comment has been minimized.

codecov bot commented Sep 10, 2018

Codecov Report

Merging #146 into master will increase coverage by 0.77%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #146      +/-   ##
==========================================
+ Coverage    95.3%   96.08%   +0.77%     
==========================================
  Files           9        9              
  Lines         597      716     +119     
  Branches       38       47       +9     
==========================================
+ Hits          569      688     +119     
  Misses         28       28
Impacted Files Coverage 螖
tests/test_util.py 100% <100%> (酶) 猬嗭笍
miss_islington/status_change.py 100% <100%> (酶) 猬嗭笍
miss_islington/backport_pr.py 100% <100%> (酶) 猬嗭笍
tests/test_status_change.py 100% <100%> (酶) 猬嗭笍
miss_islington/util.py 100% <100%> (酶) 猬嗭笍

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 85c6a4e...9d6e3e6. Read the comment docs.

@mariatta-bot

This comment has been minimized.

mariatta-bot commented Sep 10, 2018

Mariatta was mentioned, but she's out of open source for the rest of September 2018. Be aware she might not get to this until October 2018.
(I'm a bot)

@Mariatta Mariatta closed this Sep 11, 2018

@Mariatta Mariatta reopened this Sep 11, 2018

@Mariatta Mariatta closed this Sep 11, 2018

@Mariatta Mariatta reopened this Sep 11, 2018

@Mariatta Mariatta force-pushed the Mariatta:automerge-master-branch branch from ad19983 to 4a70ff3 Sep 11, 2018

Mariatta added some commits Sep 10, 2018

Automerge PR labeled with "automerge".
Merge the PR if:
- PR has "awaiting merge" and "automerge" labels, and
- all CI status checks have passed, and
- replace # with GH-
- get the commit message from PR title and description

Closes python/bedevere#14

@Mariatta Mariatta force-pushed the Mariatta:automerge-master-branch branch from cc7af3e to 87e25bc Sep 11, 2018

@Mariatta Mariatta temporarily deployed to miss-islington Sep 11, 2018 Inactive

@Mariatta Mariatta temporarily deployed to miss-islington Sep 11, 2018 Inactive

@Mariatta Mariatta temporarily deployed to miss-islington Sep 11, 2018 Inactive

@Mariatta Mariatta merged commit 3215042 into python:master Sep 11, 2018

3 checks passed

codecov/patch 100% of diff hit (target 95.3%)
Details
codecov/project 96.08% (+0.77%) compared to 85c6a4e
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Mariatta Mariatta deleted the Mariatta:automerge-master-branch branch Sep 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment