-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add support to customize working directory #1493
base: main
Are you sure you want to change the base?
Conversation
Yes! This is badly needed in our use case. |
Any chance this could get a review? |
@ericsciple this would be awesome to get added |
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.
LGTM
Hello, |
Merge |
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.
Done
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.
Merge
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.
Go gogo gog
Is there any tracking regarding this PR? |
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.
It looks all good for me. Do you know if we can get this in? We need this for our enterprise org repo's workflows.
@jww3 Please can you assist |
Related/possible duplicate issues: #955 |
Let user override $GITHUB_WORKSPACE as default working directory Defaults to undefined, the original behaviour is maintained
Add working-directory input parameter
Add working-directory
Not needed (working-directory option is used internally)
Remove path.sep during comparison with repositoryPath (did not take in account root folders) Updated and added more unit tests Updated dist/index.js
4cd8c93
to
8461fff
Compare
any update with that? its still unresolved, what is blocking merging this feature? |
Allow the users to customize the working directory where the commands of the action will be executed, e.g.: c:, /home/user, etc.
This is to allow the users to switch to a different mount point (for example github hosted windows runners offer quite a bit of space but the checkout process happens on d:, where the space is only 14GB).
Sometimes it is not feasible to copy the repository after cloning it because:
I have added a few unit tests and tested it on a windows-latest runner on this repository:
https://github.com/accodev/trackmania-manager-py/actions/runs/6263454133/job/17007891987
Note that the chosen working directory has to exists (kept the original behaviour).