Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add documentation onon.workflow_run.workflows#37022

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
EnricoMi wants to merge3 commits intogithub:main
base:main
Choose a base branch
Loading
fromEnricoMi:workflow_run-workflows

Conversation

EnricoMi
Copy link

@EnricoMiEnricoMi commentedMar 21, 2025
edited
Loading

Why:

The values ofon.workflow_run.workflows are treated as glob patterns. Special characters requires escaping.

Details:
https://github.com/orgs/community/discussions/50835
actions/runner#3763

Closes:#12572

What's being changed (if available, include any code snippets, screenshots, or gifs):

Adds documentation foron.workflow_run.workflows, especially around glob patterns.

Check off the following:

  • A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require an SME review from GitHub staff.
  • The changes in this PR meetthe docs fundamentals that are required for all content.
  • All CI checks are passing and the changes look good in the review environment.

tr4200812 and aci26c reacted with thumbs up emoji
@welcomeWelcome
Copy link

welcomebot commentedMar 21, 2025

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out thecontributing guidelines.

@github-actionsgithub-actionsbot added the triageDo not begin working on this issue until triaged by the team labelMar 21, 2025
@github-actionsGitHub Actions
Copy link
Contributor

github-actionsbot commentedMar 21, 2025
edited
Loading

How to review these changes 👓

Thank you for your contribution. To review these changes, choose one of the following options:

A Hubber will need to deploy your changes internally to review.

Table of review links

⚠️Warning: Our review server is experiencing latency issues.

The table shows the files in thecontent directory that were changed in this pull request. This helps you review your changes on the review server. Changes to thedata directory are not included in this table.

SourceReviewProductionWhat Changed
actions/writing-workflows/workflow-syntax-for-github-actions.mdfpt
ghec
ghes@3.163.153.143.133.12
fpt
ghec
ghes@3.163.153.143.133.12

Key:fpt: Free, Pro, Team;ghec: GitHub Enterprise Cloud;ghes: GitHub Enterprise Server

🤖 This comment isautomatically generated.

@EnricoMiEnricoMi changed the titleAdd documentation on workflow_run.workflowsAdd documentation onon.workflow_run.workflowsMar 21, 2025
@Sharra-writesSharra-writes added contentThis issue or pull request belongs to the Docs Content team needs SMEThis proposal needs review from a subject matter expert and removed triageDo not begin working on this issue until triaged by the team labelsMar 21, 2025
@github-actionsGitHub Actions
Copy link
Contributor

Thanks for opening a pull request! We've triaged this issue for technical review by a subject matter expert 👀

@Sharra-writes
Copy link
Contributor

Thanks so much for opening a PR! I'll get this triaged for review.

@Sharra-writes
Copy link
Contributor

Hi,@EnricoMi. 👋 One of the team wanted to make sure you had seenthis while we're waiting for feedback from our SMEs on this issue.

@EnricoMi
Copy link
Author

@Sharra-writes thanks for the reply! Please let the team know that I do not see the connection between the DockerfileWORKDIR option and how Github Actions evaluates values foron.workflow_run.workflow.

@Sharra-writes
Copy link
Contributor

Sharra-writes commentedMar 26, 2025
edited
Loading

@EnricoMi Well, no wonder it doesn't seem relevant, because I got my tabs mixed up and put this on the wrong pull request. 😅 I'm so sorry, and thank you for being patient with us. I do have this open for the SMEs to look at.

@subatoi
Copy link
Contributor

Hi@EnricoMi, thanks for raising this—I'm going to reopen the original issue so we can transfer it internally.

We'll schedule this work as soon as we can. Just so you're aware, there's a few moving parts to this and I can't guarantee that we'll use this PR: since we need to make some further changes, it may be easier for us to start a new one instead. If we do that, would you like us to add you as a co-committer? I notice you're notusingnoreply on your email address in your commits.

Many thanks

@EnricoMi
Copy link
Author

would you like us to add you as a co-committer

yes, please

thyeggman reacted with heart emoji

@Andzhela78

This comment was marked as spam.

Trickskeee3013

This comment was marked as spam.

@github-actionsGitHub Actions
Copy link
Contributor

This is a gentle bump for the docs team that this PR is waiting for technical review.

@github-actionsgithub-actionsbot added the SME staleThe request for an SME has staled labelJun 6, 2025
@Sharra-writesSharra-writes added never-staleDo not close as stale and removed SME staleThe request for an SME has staled labelsJun 6, 2025
@github-actionsGitHub Actions
Copy link
Contributor

This is a gentle bump for the docs team that this PR is waiting for technical review.

@github-actionsgithub-actionsbot added SME staleThe request for an SME has staled and removed SME staleThe request for an SME has staled labelsJul 5, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@Trickskeee3013Trickskeee3013Trickskeee3013 left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
contentThis issue or pull request belongs to the Docs Content teamneeds SMEThis proposal needs review from a subject matter expertnever-staleDo not close as stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Documentworkflow_run behaviour when workflow name contains special characters
5 participants
@EnricoMi@Sharra-writes@subatoi@Andzhela78@Trickskeee3013

[8]ページ先頭

©2009-2025 Movatter.jp