Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add github actions build #150

Merged
merged 2 commits into from
Dec 4, 2020
Merged

Add github actions build #150

merged 2 commits into from
Dec 4, 2020

Conversation

vstelmakh
Copy link
Contributor

@vstelmakh vstelmakh commented Dec 3, 2020

Regarding #141
This is simple github actions CI, running PHPUnit. Composer dependencies are cached by date. If there are no changes in dependencies - multiple builds at same day run faster.

See build example at repository fork.

uses: shivammathur/setup-php@v2
with:
php-version: '7.4'
tools: composer:v1
Copy link
Contributor Author

@vstelmakh vstelmakh Dec 3, 2020

Choose a reason for hiding this comment

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

There are some package plugins incompatibilities with composer v2 at this moment, so I restricted it to v1.

- name: Validate composer.json and composer.lock
run: composer validate

- name: Prepare Composer cache vars
Copy link
Contributor

Choose a reason for hiding this comment

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

You could simplify this steps by using https://github.com/ramsey/composer-install

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done in #152

@mnapoli
Copy link
Owner

mnapoli commented Dec 4, 2020

Thanks a lot!

I'll merge anyway because I think it's a good improvement, and if you are able to send a new PR that simplifies the Composer steps, as suggested above, that's perfect!

@mnapoli mnapoli merged commit 008bbd9 into mnapoli:master Dec 4, 2020
@vstelmakh vstelmakh deleted the add-github-actions branch December 4, 2020 14:49
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