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

moda-ci.yaml triggers an error in forks for anyone who doesn't have access to github/docs-internal #35731

Closed
jsoref opened this issue Dec 20, 2024 · 2 comments
Labels
content This issue or pull request belongs to the Docs Content team fix-internally Triggers a workflow to copy the issue internally and close the current issue

Comments

@jsoref
Copy link
Contributor

jsoref commented Dec 20, 2024

This guard

if: ${{ github.repository == 'github/docs-internal' }}

did not prevent:
Invalid workflow file: .github/workflows/moda-ci.yaml#L86

error parsing called workflow
".github/workflows/moda-ci.yaml"
-> "github/internal-actions/.github/workflows/docker_security.yml@main"
: workflow was not found. See https://docs.github.com/actions/learn-github-actions/reusing-workflows#access-to-reusable-workflows for more information.

I think it might be possible to avoid this structure by using a combination of a workflow_call with a jobs: / if: condition that guards against the user from tripping on the problem e.g. ${{ github.repository == 'github/docs-internal' }} and then have the on: workflow_call on the internal side be responsible for using the reusable workflow.

Originally posted by @jsoref in #34562 (comment)

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Dec 20, 2024
@nguyenalex836
Copy link
Contributor

@jsoref Thank you for raising the flag on this! ✨ Given the referenced file, I'll create an issue to have this reviewed internally 💛

I'll close this issue, but if you find anything else that might be relevant, feel free to ping me here and I'll copy over to the internal issue!

@nguyenalex836 nguyenalex836 added content This issue or pull request belongs to the Docs Content team fix-internally Triggers a workflow to copy the issue internally and close the current issue and removed triage Do not begin working on this issue until triaged by the team labels Dec 20, 2024
@docs-bot
Copy link
Collaborator

Thank you for opening this issue! Updates to this documentation must be made internally. I have copied your issue to an internal issue, so I will close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team fix-internally Triggers a workflow to copy the issue internally and close the current issue
Projects
None yet
Development

No branches or pull requests

3 participants