Skip to content

Create .travis.yml - #2

Closed
Mariatta wants to merge 5 commits into
masterfrom
Mariatta-patch-2
Closed

Mariatta wants to merge 5 commits into
masterfrom
Mariatta-patch-2

Conversation

@Mariatta

Copy link
Copy Markdown
Member

No description provided.

Comment thread .travis.yml Outdated
Comment thread .travis.yml
- &run-if-tagged
if: tag IS present
- &run-if-cherry-picker
if: tag =~ ^cherry\-picker\-v\d+\.\d+\.\d+$

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This can be hugely simplified now

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Can you suggest how? I'm not actually very well-versed in config file like this 😅

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just use that other anchor without a regex

Comment thread .travis.yml Outdated
Co-Authored-By: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
Comment thread .travis.yml Outdated
@webknjaz

Copy link
Copy Markdown
Member

Now that there's no blurb, script+install could be moved to the root level and most of the anchors aren't needed so the config could be ~two times smaller.

Co-Authored-By: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
@webknjaz

webknjaz commented Jun 1, 2019

Copy link
Copy Markdown
Member

If you want, I can take care of simplifying this config.

@Mariatta

Mariatta commented Jun 1, 2019

Copy link
Copy Markdown
Member Author

If you want, I can take care of simplifying this config.

Yes! I'll appreciate this. Thanks!

@webknjaz webknjaz mentioned this pull request Jun 4, 2019
@webknjaz

webknjaz commented Jun 4, 2019

Copy link
Copy Markdown
Member

#11 should do

@Mariatta Mariatta closed this Jun 5, 2019
@hugovk
hugovk deleted the Mariatta-patch-2 branch May 5, 2023 14:31
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.

4 participants