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

Travis CI: Corrected astyle job behavior with non-master base branch PRs #9346

Merged
merged 1 commit into from
Jan 11, 2019

Conversation

cmonr
Copy link
Contributor

@cmonr cmonr commented Jan 11, 2019

Description

Consolidated efforts from PR #9336.

Previous job's env vars would not be populated correctly if the base branch of a PR was not master.
Corected by pulling remaining respository information to perform comparison between read-only instance of PR and base branch.

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Consolidated efforts from PR ARMmbed#9336.

Previous job's env vars would not be populated correctly if the base branch of a PR was not master.
Corected by pulling remaining respository information to perform comparison between read-only instance of PR and base branch.
@cmonr cmonr requested a review from a team January 11, 2019 04:35
@cmonr cmonr changed the title Corrected simplified astyle job behavior when base branch is not master Travis CI: Corrected simplified astyle job behavior with non-master base branch PRs Jan 11, 2019
@cmonr cmonr changed the title Travis CI: Corrected simplified astyle job behavior with non-master base branch PRs Travis CI: Corrected astyle job behavior with non-master base branch PRs Jan 11, 2019
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 11, 2019

@kjbracey-arm Can you review this one ?

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 11, 2019

CI started

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.

None yet

2 participants