Laravel-Userstamps icon indicating copy to clipboard operation
Laravel-Userstamps copied to clipboard

Published 20 hours ago • verified publisher icon WildsideUK

Migrate to github actions

Open kosarinin opened this issue 3 years ago • 3 comments

Hey @mattmcdonald-uk Thank you for creating this package!

I've noticed this package lost its ci build. I assume because of https://earthly.dev/blog/migrating-from-travis/

In this PR the following changes were introduced:

  • Removed travis ci configuration file
  • Travis ci replaced by github actions
  • Updated "Build" badge in the README.md

Thank you!

kosarinin avatar Feb 05 '22 20:02 kosarinin

Have you considered defining the matrix using a shorter form to save a few lines?

https://github.com/SynergiTech/laravel-postal/blob/aa47a609b27560d8657f2ffd071623aa1cb5ad39/.github/workflows/tests.yml#L10-L17

willpower232 avatar Feb 13 '22 15:02 willpower232

Hey @willpower232 Yes, I considered using a "shorter form" but if we need to match all the requirements from the travis configuration we will have to write the long list of exclude statements which is the same...

kosarinin avatar Feb 19 '22 14:02 kosarinin

Is there any specific ETA for this PR?

Nks avatar Apr 20 '22 18:04 Nks