Skip to content

Conversation

@mhaas
Copy link
Contributor

@mhaas mhaas commented Jul 22, 2020

Travis does not retain any files between different stages (or jobs).
Because the traceability report is generated during the system test stage,
this commit moves the deploy steps into the system test stage where the
file is actually available.

mhaas added 2 commits July 22, 2020 17:07
Travis does not retain any files between different stages (or jobs).
Because the traceability report is generated during the system test stage,
this commit moves the deploy steps into the system test stage where the
file is actually available.
@mhaas mhaas requested review from EvgenyArn and karthikmswamy July 22, 2020 15:12
Copy link
Contributor

@karthikmswamy karthikmswamy left a comment

Choose a reason for hiding this comment

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

Looks good to me. I just have a minor comment about the naming and os.

condition: $TRAVIS_TAG =~ ^rel/.*$
skip_existing: true
- stage: test
name: "Python: 3.8 on Windows"
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm a bit confused about name being Python 3.8 on Windows when the os below is osx. What am I missing here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That is actually two different stages - you were probably looking at the collapsed view in the diff. Does it make more sense when you view the full file?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh yes, now I see it. Thanks for adding that!

@mhaas mhaas merged commit e5642e1 into master Jul 23, 2020
@mhaas mhaas deleted the chore/travis_fix_traceability_upload branch July 23, 2020 07:25
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.

3 participants