Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[CI] Switch packages build from Travis to GitHub Actions#12089

Merged
pamil merged 1 commit into
Sylius:1.8from
pamil:1.8-packages-github-actions
Nov 26, 2020
Merged

[CI] Switch packages build from Travis to GitHub Actions#12089
pamil merged 1 commit into
Sylius:1.8from
pamil:1.8-packages-github-actions

Conversation

@pamil
Copy link
Copy Markdown
Contributor

@pamil pamil commented Nov 25, 2020

This will create two jobs for every package that we have, making use of 20 parallel jobs GitHub Actions provides.

@pamil pamil requested a review from a team as a code owner November 25, 2020 15:25
@probot-autolabeler probot-autolabeler Bot added the Maintenance CI configurations, READMEs, releases, etc. label Nov 25, 2020
@pamil pamil merged commit 0286ff8 into Sylius:1.8 Nov 26, 2020
@pamil pamil deleted the 1.8-packages-github-actions branch November 26, 2020 08:45
pamil added a commit that referenced this pull request Nov 30, 2020
…ctions (pamil)

This PR was merged into the 1.8 branch.

Discussion
----------

The last PR of the three-part series (#12087, #12089) - switching the application build from Travis CI to GitHub Actions.

Non-JS Behat take around 20% longer (from ~10-11 minutes to ~12-14 minutes) and PHPUnit around 50% longer (from ~1.5m to ~2.5m), I've got no idea what is it caused by, but we've much better parallelization of the builds, so we should be able to get feedback much faster anyway.

Commits
-------

1139d72 [CI] Switch application build from Travis to GitHub Actions
if: matrix.symfony != ''
run: |
composer global require --no-progress --no-scripts --no-plugins "symfony/flex:^1.10"
composer config extra.symfony.require "${{ matrix.symfony }}"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

shouldn't this have working-directory too, to set this extra.symfony.require config in the right composer.json ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thanks!

pamil added a commit that referenced this pull request Dec 2, 2020
This PR was merged into the 1.8 branch.

Discussion
----------

Thanks to @stof comments in #12090 and #12089.

Commits
-------

ef06041 [CI] Run all static checks even if one of them fails
35b4820 [CI] Fix requiring specific Symfony version in packages build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance CI configurations, READMEs, releases, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants