Skip to content

Conversation

localheinz
Copy link
Contributor

This PR

  • adds a very small workflow installing root dependencies with composer using GitHub Actions

💁‍♂ This could be a small start, and we could switch over more steps as seen in #191.

@localheinz localheinz requested a review from Kharhamel January 29, 2020 16:15
@localheinz localheinz force-pushed the feature/small branch 2 times, most recently from 6e2e588 to dcb1a9b Compare January 30, 2020 09:57
@Kharhamel
Copy link
Collaborator

Sorry, I didn't reply sooner, I had quite a lot of work lately. Thanks for this PR. I am going to test this on my own fork before merging, I will try to do this this week.

@localheinz
Copy link
Contributor Author

@Kharhamel

You can take a look at this workflow in action at https://github.com/localheinz/safe/pull/1.

@Kharhamel Kharhamel merged commit 59de914 into thecodingmachine:master Feb 14, 2020
@Kharhamel
Copy link
Collaborator

Ok this looks good, thank you very much

@localheinz
Copy link
Contributor Author

Thank you, @Kharhamel!

@localheinz localheinz deleted the feature/small branch February 14, 2020 18:24
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.

2 participants