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

ci: use github action for bulk of github test #31

Merged
merged 2 commits into from
Sep 17, 2023

Conversation

rfay
Copy link
Member

@rfay rfay commented Aug 28, 2023

The Issue

We now have the lovely github-action-add-on-test from @julienloizelet - THANKS!

This uses it.

@rfay rfay requested a review from tyler36 August 28, 2023 18:46
@rfay
Copy link
Member Author

rfay commented Aug 28, 2023

Please confirm correctness @julienloizelet @tyler36

I also implemented this in ddev-platformsh

@rfay rfay force-pushed the 20230828_use_github_action branch from 810a26b to f1d038d Compare August 28, 2023 20:01
@julienloizelet
Copy link
Contributor

Thanks @rfay ,

We already set bash as shell in every step of the action because it is required to define one : https://github.com/ddev/github-action-add-on-test/blob/main/action.yaml#L63 for example

The DDEV_GITHUB_TOKEN value is already set in the action: https://github.com/ddev/github-action-add-on-test/blob/main/action.yaml#L98-L100

There's no harm in keeping it, but :

  • the fewer things there are outside the action, the less maintenance is required
  • If someone set the DDEV_GITHUB_TOKEN value outside the action, it will be overridden in the action by the token input value.
  • Everyday, this run https://github.com/ddev/github-action-add-on-test/actions/workflows/add-ons-test.yml launches about 20 quasi simultaneous ddev get without reaching the API rate limit (It doesn't prove anything, but it suggests that it works.).

@@ -40,12 +40,10 @@ A repository like this one is the way to get started. You can create a new repo
8. Create a release on GitHub.
9. Test manually with `ddev get <owner/repo>`.
10. You can test PRs with `ddev get https://github.com/<user>/<repo>/tarball/<branch>`
11. Update the README.md to describe the add-on, how to use it, and how to contribute. If there are any manual actions that have to be taken, please explain them. If it requires special configuration of the using project, please explain how to do those. Examples in [ddev/ddev-drupal9-solr](https://github.com/ddev/ddev-drupal9-solr), [ddev/ddev-memcached](github.com/ddev/ddev-memcached), and [ddev/ddev-beanstalkd](https://github.com/ddev/ddev-beanstalkd).
11. Update the README.md to describe the add-on, how to use it, and how to contribute. If there are any manual actions that have to be taken, please explain them. If it requires special configuration of the using project, please explain how to do those. Examples in [ddev/ddev-solr](https://github.com/ddev/ddev-solr), [ddev/ddev-memcached](github.com/ddev/ddev-memcached), and (advanced) [ddev-platformsh](https://github.com/ddev/ddev-platformsh).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
11. Update the README.md to describe the add-on, how to use it, and how to contribute. If there are any manual actions that have to be taken, please explain them. If it requires special configuration of the using project, please explain how to do those. Examples in [ddev/ddev-solr](https://github.com/ddev/ddev-solr), [ddev/ddev-memcached](github.com/ddev/ddev-memcached), and (advanced) [ddev-platformsh](https://github.com/ddev/ddev-platformsh).
11. Update the README.md to describe the add-on, how to use it, and how to contribute. If there are any manual actions that have to be taken, please explain them. If it requires special configuration of the using project, please explain how to do those. Examples in [ddev/ddev-solr](https://github.com/ddev/ddev-solr), [ddev/ddev-memcached](github.com/ddev/ddev-memcached), and [ddev-platformsh](https://github.com/ddev/ddev-platformsh).

Not sure if you want to include advance or it is left over

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was intentional, it's a hard one to understand, but demonstrates the most exotic things I know to do with an add-on. Probably too exotic.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was intentional

Thank you for clarifying. Looks good to me.

@rfay
Copy link
Member Author

rfay commented Aug 29, 2023

See what you think now @julienloizelet - thanks for paying careful attention.

@rfay rfay force-pushed the 20230828_use_github_action branch from 61e15cd to 6ce4606 Compare September 17, 2023 02:34
@rfay rfay merged commit 8c39807 into ddev:main Sep 17, 2023
@rfay rfay deleted the 20230828_use_github_action branch September 17, 2023 02:44
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.

3 participants