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

Update Github Actions #23

Conversation

RoussKS
Copy link

@RoussKS RoussKS commented Feb 29, 2024

Update github actions PR

Nothing really important, but felt like doing a minor contribution after ages 😄

  • Update to use checkout v4 github action
  • Use PHP matrix to perform static analysis & tests to PHP version 7.3, 7.4, 8.1, 8.2
  • Add installable tools to php setup github action
  • Add cache github action to save & load vendor packages from cache instead of re-installing
  • Indentation to phpunit.xml.dist file

@RoussKS RoussKS closed this Feb 29, 2024
@RoussKS RoussKS changed the title update github actions to latest checkout versions & use cache compone… Update Github actions PR Feb 29, 2024
@RoussKS RoussKS changed the title Update Github actions PR Update Github Actions Feb 29, 2024
@RoussKS
Copy link
Author

RoussKS commented Feb 29, 2024

Re-opening PR after local repo test was completed

@RoussKS RoussKS reopened this Feb 29, 2024
@RoussKS
Copy link
Author

RoussKS commented Apr 24, 2024

@aashwin-rvvup not sure if you had the chance to have a look at this?

Let me know if you want to do it against a different branch to allow for the pipelines to work

@aashwin-rvvup aashwin-rvvup changed the base branch from main to RoussKS-update-github-actions April 25, 2024 08:02
@aashwin-rvvup
Copy link
Collaborator

@aashwin-rvvup not sure if you had the chance to have a look at this?

Let me know if you want to do it against a different branch to allow for the pipelines to work

Hi Rou, thanks for your contribution I missed this PR previously. I have updated the base branch to a local one in this repo so we can check the pipelines. Ill take a look at this PR and get it in either today or Monday!

@aashwin-rvvup aashwin-rvvup merged commit 95073ce into rvvup:RoussKS-update-github-actions Apr 25, 2024
@RoussKS RoussKS deleted the update-github-actions branch May 24, 2024 07:14
@RoussKS RoussKS restored the update-github-actions branch May 24, 2024 07:22
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