Skip to content

correctly determine commit status in --merge-pr#3406

Merged
akesandgren merged 1 commit intoeasybuilders:developfrom
boegel:det_commit_status_travis_github_actions
Aug 17, 2020
Merged

correctly determine commit status in --merge-pr#3406
akesandgren merged 1 commit intoeasybuilders:developfrom
boegel:det_commit_status_travis_github_actions

Conversation

@boegel
Copy link
Member

@boegel boegel commented Aug 15, 2020

fixes #3405

@boegel
Copy link
Member Author

boegel commented Aug 16, 2020

waves fist at Travis CI

@boegel boegel closed this Aug 16, 2020
@boegel boegel reopened this Aug 16, 2020
@boegel boegel closed this Aug 16, 2020
@boegel boegel reopened this Aug 16, 2020
@easybuilders easybuilders deleted a comment from boegelbot Aug 16, 2020
@boegel boegel closed this Aug 16, 2020
@boegel boegel reopened this Aug 16, 2020
@easybuilders easybuilders deleted a comment from boegelbot Aug 16, 2020
@boegel boegel force-pushed the det_commit_status_travis_github_actions branch from ab8cbd1 to 479cf33 Compare August 16, 2020 16:43
@easybuilders easybuilders deleted a comment from boegelbot Aug 16, 2020
@smoors
Copy link
Contributor

smoors commented Aug 16, 2020

tested, seems to work well

@easybuilders easybuilders deleted a comment from boegelbot Aug 17, 2020
Copy link
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@akesandgren
Copy link
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit f8e4958 into easybuilders:develop Aug 17, 2020
@boegel boegel deleted the det_commit_status_travis_github_actions branch August 17, 2020 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--merge-pr refuses to merge easyconfig PRs because test suite is "pending"

3 participants