-
Notifications
You must be signed in to change notification settings - Fork 34
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
[sc-157321] Allow designating dir
in github action
#294
[sc-157321] Allow designating dir
in github action
#294
Conversation
This pull request has been linked to Shortcut Story #157321: github action should allow |
dir: | ||
description: "Path to existing checkout of the repository. You may use this if your action checkouts multiple repositories. The default value is `$GITHUB_WORKSPACE`." | ||
required: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't set an explicit default sting here, because it reads the default from environment variable $GITHUB_WORKSPACE
8fdabc6
to
bf6a0a9
Compare
@mmrj i had to change how this works, sorry to make ya re-review! |
@InTheCloudDan wasn't sure how to test this since, if you have thoughts i'd love to hear them. |
@jazanne yeah this is an interesting one, I'm not really sure what would be the best way to it. Maybe just a single unit test confirming what's set is correctly passed in?? |
Co-authored-by: Molly <molly.jones@launchdarkly.com>
Co-authored-by: Molly <molly.jones@launchdarkly.com>
13303a4
to
05e66d4
Compare
not high priority |
This is completely optional, but is useful if someone checks out multiple repos in their action.
For example, this git-gatsby workflow used to check out multiple repos internal repos
--
Dev notes
this must use
env
override and not github action input because of how env variable is used to find the configuration file etc.