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

Support a custom working directory #21

Merged
merged 5 commits into from
Sep 10, 2023
Merged

Conversation

qubitz
Copy link
Contributor

@qubitz qubitz commented Jul 10, 2023

This change allows users to optionally specify which directory has the git repository.

In rare cases, the directory containing the desired repository does not exist within the github workspace on a runner. When this occurs, this action is no longer useable.

The working-directory option within a workflow is only supported for run steps and not uses:
https://stackoverflow.com/questions/67299058/running-github-actions-for-uses-in-another-directory

I am fairly new to TypeScript and would welcome any feedback.

This change allows users to optionally specify
which directory has the git repository.

In rare cases, the directory containing the
desired repository does not exist within
the github workspace on a runner. When this
occurs, this action is no longer useable.

The `working-directory` option within a workflow
is only supported for `run` steps and not `uses`:
https://stackoverflow.com/questions/67299058/running-github-actions-for-uses-in-another-directory
action.yml Show resolved Hide resolved
src/action.ts Show resolved Hide resolved
@EndBug
Copy link
Owner

EndBug commented Jul 12, 2023

Nice idea! When you're ready, mark the PR as "Ready for review" and I'll take a look at it ;)

@qubitz qubitz marked this pull request as ready for review July 12, 2023 15:09
@qubitz
Copy link
Contributor Author

qubitz commented Aug 31, 2023

@EndBug have you had a chance to look at this?

Copy link
Owner

@EndBug EndBug left a comment

Choose a reason for hiding this comment

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

Hi! Sorry but I didn't receive the notification when you marked the PR as ready for review 😅

Anyway, just looked at it and it's great! I just have one note about the setupUser function, can you take a look at the review comment?

As soon as that's fixed, I'll test the action, marge the PR, and release a new minor version 🚀

action.yml Show resolved Hide resolved
src/action.ts Show resolved Hide resolved
src/action.ts Outdated Show resolved Hide resolved
Co-authored-by: Federico Grandi <fgrandi30@gmail.com>
@qubitz
Copy link
Contributor Author

qubitz commented Sep 5, 2023

Okay, I think these changes are ready to go. TIA for testing it 👍.

Copy link
Owner

@EndBug EndBug left a comment

Choose a reason for hiding this comment

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

Alright, LGTM ✅

@EndBug EndBug merged commit 9cdefcf into EndBug:main Sep 10, 2023
2 checks passed
@EndBug
Copy link
Owner

EndBug commented Sep 10, 2023

@all-contributors please add @qubitz for their code contribution ✨

@allcontributors
Copy link
Contributor

@EndBug

I've put up a pull request to add @qubitz! 🎉

@EndBug
Copy link
Owner

EndBug commented Sep 10, 2023

Released in v1.6.0 (also v1 and latest) 🚀
Thanks for the help ❤️

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.

None yet

2 participants