This repository has been archived by the owner on Jun 15, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2
Migrate emteknetnz gha actions #36
Comments
This was referenced May 31, 2022
This was referenced May 31, 2022
Closed
@emteknetnz Can you please add a meaningful description to each of the gha repositories? It will make it a lot easier to know at a glance what each repo is for. |
Descriptions are in the readme.md of each pull-request |
6 months down the track, nobody will want to open each repo to remember, for example, the distinction between |
Yes, that's the one. Sorry, I should have been more explicit. |
Have added descriptions on the silverstripe account repos |
This was referenced Jun 8, 2022
Closed
@emteknetnz PRs merged. I think the next step is for you to create a |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Create new repos on silverstripe account with emtpy readme only, then fork repos to emteknetnz, and create PRs with copy pasted contents from gha versions so they can all be peer reviewed
I'm thinking we should retain the gha- prefix instead of the github-actions- prefix, and also split the exsiting 'workflow_call' style github-actions-ci-ci repo to a pair of actions to be consistent:
ACs
PRs
The following will be handled with silverstripe/gha-ci#4:
MNT Rename test class to conform with standard naming convention silverstripe-assets#496MNT Rename test class to conform with standard naming convention silverstripe-framework#10353MNT Rename test class to conform with standard naming convention (v3) silverstripe-graphql#472MNT Rename test class to conform with standard naming convention (v4) silverstripe-graphql#473The text was updated successfully, but these errors were encountered: