Skip to content

travis: add after_script step to check warnings#214

Closed
marc-hb wants to merge 1 commit intothesofproject:masterfrom
marc-hb:check-warnings
Closed

travis: add after_script step to check warnings#214
marc-hb wants to merge 1 commit intothesofproject:masterfrom
marc-hb:check-warnings

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Apr 4, 2020

Signed-off-by: Marc Herbert marc.herbert@intel.com

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb requested review from deb-intel and xiulipan April 4, 2020 07:10
@marc-hb marc-hb changed the title travis: add after_success step to check warnings travis: add after_script step to check warnings Apr 4, 2020
@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 4, 2020

https://docs.travis-ci.com/user/job-lifecycle#breaking-the-build

The exit code of after_success, after_failure, after_script, after_deploy and subsequent stages do not affect the build result.

Yet it's red above.

I'm adding after_script instead of script so deploy still runs even when after_script fails. Correct?

There's apparently no intermediate, "yellow" state in Travis, it's either green or red.

@marc-hb
Copy link
Collaborator Author

marc-hb commented Apr 16, 2020

I have a newer and much better version of this, I will submit it after #225 and thesofproject/sof#2773 are merged.

@marc-hb marc-hb closed this Apr 16, 2020
@marc-hb marc-hb deleted the check-warnings branch April 16, 2020 20:53
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