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

feat: add input for module path #15

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

yann-soubeyrand
Copy link
Member

@yann-soubeyrand yann-soubeyrand commented May 31, 2024

Description

This pull request adds a new module-path input to the action to support Dagger modules located elsewhere than in the current working directory.

Motivation

When testing Daggerverse repositories containing several Dagger modules, one can use a GitHub Actions job with a matrix strategy to test each module individually. In this case, it’s necessary to be able to pass the module path to this action.

Linked issues

Checklist

  • This pull request:

    • either fixes an issue (please refer to the security policy before submitting a security fix),
    • or implements a requested feature which has been discussed with the team first.
  • If applicable, I’ve added the required tests.

  • If applicable, I’ve done the necessary documentation changes.

  • I’m legally entitled to submit this pull request and I’ve signed off all my commits to certify that my contribution conforms to the Developer Certificate of Origin (DCO).

@yann-soubeyrand yann-soubeyrand requested a review from a team as a code owner May 31, 2024 08:33
@yann-soubeyrand yann-soubeyrand force-pushed the feat/add-module-path-input branch 4 times, most recently from fb06842 to 35d232b Compare May 31, 2024 08:57
Signed-off-by: Yann Soubeyrand <[email protected]>
@yann-soubeyrand yann-soubeyrand merged commit 1d2b42a into main May 31, 2024
7 checks passed
@yann-soubeyrand yann-soubeyrand deleted the feat/add-module-path-input branch May 31, 2024 09:07
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.

1 participant