Skip to content

Travis test removed file for astyle #9351

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

Closed
wants to merge 4 commits into from

Conversation

0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Jan 11, 2019

Description

Testing one use case that might cause Travis to fail (astyle job).

Pull request type

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

Reviewers

Cruz Monrreal II and others added 2 commits January 10, 2019 22:28
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.
@0xc0170 0xc0170 changed the title test removed file Travis test removed file for astyle Jan 11, 2019
@0xc0170
Copy link
Contributor Author

0xc0170 commented Jan 11, 2019

travis-ci/astyle — Local astyle testing has passed - confirmed the #9346 fixing also deleted files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant