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

[Feature request] Support reporting to a PR from another repository #25

Open
lachaib opened this issue Oct 16, 2023 · 0 comments · May be fixed by #26
Open

[Feature request] Support reporting to a PR from another repository #25

lachaib opened this issue Oct 16, 2023 · 0 comments · May be fixed by #26

Comments

@lachaib
Copy link

lachaib commented Oct 16, 2023

Hello,

In my current organisation, we've got a specific repository (say robot-tests) containing robot framework tests and corresponding secrets. In order to run those tests after deployment of any service stored in other repositories (serviceA, serviceB), I need to dispatch a workflow. I insist on the workflow_dispatch event because the workflow_call event from another repo cannot access secrets stored for robot-tests repository (and it's a bad practice to store everything at organisation level).

Therefore, I would like to enable this action to support passing another repository as input.

Following what I could see in the action code, the simplest way would be to add

repository:
    type: string
    description: Repository ([Owner/]Repository) of pull request to decorate with report
    default: "${{github.repository}}"

and modify the bash step to parse it instead of direct parsing of ${{github.repository}}

I will make a PR accordingly if you agree with it.

lachaib added a commit to lachaib/robotframework-reporter-action that referenced this issue Oct 16, 2023
@lachaib lachaib linked a pull request Oct 16, 2023 that will close this issue
lachaib added a commit to lachaib/robotframework-reporter-action that referenced this issue Oct 16, 2023
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 a pull request may close this issue.

1 participant