-
Notifications
You must be signed in to change notification settings - Fork 3k
Travis: Move frozen tools check to the end #14665
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
Conversation
@LDong-Arm, thank you for your changes. |
CI started |
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
@LDong-Arm looks like you have a conflict which needs resolving |
When the scripts for the old tools are modified (e.g. when a new target is added), the frozen tools check fails but we allow the change if it's really needed. By moving the frozen tools check to the end, Travis doesn't terminate early and the other tests get to run.
2807f88
to
f9a0501
Compare
Pull request has been modified.
Rebased now. Waiting for Travis to complete, |
CI started |
Summary of changes
When the scripts for the old tools are modified (e.g. when a new target is added), the frozen tools check fails but we allow the change if it's really needed. By moving the frozen tools check to the end, Travis doesn't terminate early and the other tests get to run.
Impact of changes
Migration actions required
Documentation
None.
Pull request type
Test results
Reviewers
@ARMmbed/mbed-os-maintainers