Skip to content

Enhancement: Use stages on Travis#174

Merged
lcobucci merged 1 commit intodoctrine:masterfrom
localheinz:feature/stages
Feb 3, 2018
Merged

Enhancement: Use stages on Travis#174
lcobucci merged 1 commit intodoctrine:masterfrom
localheinz:feature/stages

Conversation

@localheinz
Copy link
Copy Markdown
Contributor

@localheinz localheinz commented Feb 3, 2018

This PR

  • uses build stages on Travis

Blocks #177.

💁‍♂️ For reference, see https://docs.travis-ci.com/user/build-stages/.

@Majkl578
Copy link
Copy Markdown
Contributor

Majkl578 commented Feb 3, 2018

Can you please use same setup/layout as we use in other Doctrine repos where Stages are already introduced? I don't really like having different setups for different projects... Thanks.

@localheinz localheinz force-pushed the feature/stages branch 2 times, most recently from 2ae536c to 948d1b9 Compare February 3, 2018 17:02
@localheinz
Copy link
Copy Markdown
Contributor Author

@Majkl578

Sure, using doctrine/migrations as a basis then, although I have to admit I don't find it very readable.

@Majkl578
Copy link
Copy Markdown
Contributor

Majkl578 commented Feb 3, 2018

I have to admit I don't find it very readable.

The same could be said about the proposed change here - i.e. I've never seen & or* in the config. I'm not opposed to improvements, but let's keep it consistent if possible and propose changes afterwards.

@localheinz
Copy link
Copy Markdown
Contributor Author

@Majkl578

I can recommend http://blog.daemonl.com/2016/02/yaml.html, then! 🤓

Copy link
Copy Markdown
Member

@lcobucci lcobucci left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

Comment thread .travis.yml
include:
- stage: Lint
before_script:
- travis_retry composer require --dev --prefer-dist --prefer-stable phpstan/phpstan:^0.7
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.

We need to upgrade phpstan to ^0.9!

@lcobucci lcobucci self-assigned this Feb 3, 2018
@lcobucci lcobucci added this to the 1.7.0 milestone Feb 3, 2018
@lcobucci lcobucci merged commit ccb1edc into doctrine:master Feb 3, 2018
@lcobucci
Copy link
Copy Markdown
Member

lcobucci commented Feb 3, 2018

@localheinz 🚢 thanks!

@localheinz localheinz deleted the feature/stages branch February 3, 2018 17:43
@localheinz
Copy link
Copy Markdown
Contributor Author

Thank you, @lcobucci and @Majkl578!

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.

3 participants