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 action to build and test #35

Merged
merged 3 commits into from
Jul 16, 2024
Merged

Conversation

mchv
Copy link
Member

@mchv mchv commented Apr 15, 2024

What does this change?

  • Add build.yml to build and run tests on each PR
  • Created a Fastly test service for running tests
  • Created a Fastly automation token to use for the tests
  • Stored Fastly service id and token as repository secrets
  • Configure build.yml to use secrets instead of local config file

Once those changes are merged, #34 can be rebased and merged safely.

@mchv mchv requested a review from a team as a code owner April 15, 2024 13:58
@mchv mchv self-assigned this Apr 15, 2024
@mchv
Copy link
Member Author

mchv commented Apr 15, 2024

This can now be reviewed, after merging next steps would be:

Closes this ticket: #32
Also, I would like to run the build again in this branch because it hasn't ran for a while and it looks like `workflow_dispatch` will only show when the code is on main: https://stackoverflow.com/questions/67523882/workflow-is-not-shown-so-i-cannot-run-it-manually-github-actions
@ioannakok ioannakok merged commit 3cb41e6 into main Jul 16, 2024
2 checks passed
@mxdvl mxdvl deleted the mc-update-dependencies branch July 16, 2024 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants