Skip to content

Conversation

@xakraz
Copy link

@xakraz xakraz commented Oct 30, 2024

No description provided.

@ryancyq
Copy link
Owner

ryancyq commented Nov 4, 2024

@xakraz thank you the contribution. Could you please describe the use cases in the PR as well.

The owner and repo selection part is quite straightforward but the CWD is a little confusing. The workdir switching should be done before invoking the action and the "workspace" input should be sufficient for action execution in the nested directory.

@xakraz
Copy link
Author

xakraz commented Nov 5, 2024

Hi @ryancyq , thank you for reviewing and considering my PR 🙏🏻

Our current use case is the following:

  • A team maintains a project as a private repository of our organization with some sources and other content
  • During their "release" process, they want to publish some sources to another public repository
  • To do so they:
    • Trigger a GH Workflow with a tag push
    • The workflow clones/checkouts the "source" private repository related to the tag push event
    • It builds some other sources
    • It clones/checkout the "target" public repository in a specific directory within the private repository GH workflow workspace
    • Copy the built files to the checkout public repository directory
    • Commit + tag + push the changes on the "target" public repository

At first, I tried to use the "workspace" input parameter of your GHA but I didn't manage to make it work as expected.
I faced issues with the files filter and getBlob() functions if I remember correctly.

Or maybe I missed something 😅


What do you mean by ?

The workdir switching should be done before invoking the action

@xakraz
Copy link
Author

xakraz commented Dec 24, 2024

Hi @ryancyq 👋🏻

Any considerations regarding my PR and answer?

Happy holiday season ❄️ 🎄

xakraz and others added 3 commits October 3, 2025 14:00
* Update repo.ts

Signed-off-by: Xavier Krantz <xavier.krantz@datadome.co>

* fix: missing )

* update repo.ts

* fixup: build dist

* fix: return "" on tags

* chore(readme): Update inputs

---------

Signed-off-by: Xavier Krantz <xavier.krantz@datadome.co>
Co-authored-by: Stessy Mougeot <stessy.mougeot@icloud.com>
@xakraz
Copy link
Author

xakraz commented Oct 3, 2025

Hi @ryancyq 👋🏻

I have updated my PR.
What should I do to make it merged?

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.

2 participants