Skip to content
This repository has been archived by the owner on Aug 8, 2024. It is now read-only.

Set up a GitHub Actions workflow #1

Closed
phenaproxima opened this issue Mar 25, 2024 · 3 comments · Fixed by #5
Closed

Set up a GitHub Actions workflow #1

phenaproxima opened this issue Mar 25, 2024 · 3 comments · Fixed by #5
Assignees
Labels
tests Needs test coverage, or related to CI
Milestone

Comments

@phenaproxima
Copy link
Owner

This project will be tested using GitHub Actions.

We'll need to set up a workflow that builds the project (basically a composer install). The workflow should do that, and:

  • Test against a matrix of PHP 8.1, 8.2, and 8.3
  • Also against Ubuntu, macOS, and Windows.
  • The web directory should be served by Apache or nginx (we can't use the PHP built-in server because it's single-threaded).
  • Run automatically every 12 hours, and when it's triggered manually.
@phenaproxima phenaproxima added the tests Needs test coverage, or related to CI label Mar 26, 2024
@phenaproxima phenaproxima added this to the DrupalCon milestone Mar 26, 2024
@TravisCarden
Copy link
Collaborator

@phenaproxima, I can give you a PR that does all of this on Linux only that you can merge today. But macOS and Windows are taking a lot longer to figure out. Would you like an 80% value PR now and the rest in maybe a week? Or do you want to wait to get it all when I'm done?

@phenaproxima
Copy link
Owner Author

Let's take the 80%! Can you just open a follow-up issue to add support for Windows and macOS too?

@TravisCarden TravisCarden linked a pull request Apr 11, 2024 that will close this issue
9 tasks
@TravisCarden
Copy link
Collaborator

Okay, @phenaproxima; ^it's ready for you!

@phenaproxima phenaproxima modified the milestones: DrupalCon, GA Apr 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests Needs test coverage, or related to CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants