Skip to content

Actions: Automatically add language-specific labels to pull requests.#2521

Merged
RasmusWL merged 3 commits intogithub:masterfrom
tausbn:automatically-label-pull-requests
Dec 12, 2019
Merged

Actions: Automatically add language-specific labels to pull requests.#2521
RasmusWL merged 3 commits intogithub:masterfrom
tausbn:automatically-label-pull-requests

Conversation

@tausbn
Copy link
Copy Markdown
Contributor

@tausbn tausbn commented Dec 12, 2019

This seems like a cool feature, so I went ahead and created a configuration file for all of the languages in this repo. Also adds the documentation label to all PRs that modify .qhelp and .md files.
See more about this feature here:

https://github.com/actions/labeler/blob/master/README.md

(Ping @hvitved @jbj @aschackmull @max-schaefer @felicitymay)

max-schaefer
max-schaefer previously approved these changes Dec 12, 2019
Comment thread .github/labeler.yml
felicitymay
felicitymay previously approved these changes Dec 12, 2019
Copy link
Copy Markdown
Contributor

@felicitymay felicitymay left a comment

Choose a reason for hiding this comment

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

Thanks for the update

@RasmusWL RasmusWL merged commit 4330150 into github:master Dec 12, 2019
@aschackmull
Copy link
Copy Markdown
Contributor

Something appears to not be working:

##[error]YAMLException: unidentified alias "*/*.qhelp" at line 22, column 15:
      - **/*.qhelp
                  ^
##[error]unidentified alias "*/*.qhelp" at line 22, column 15:
      - **/*.qhelp
                  ^
##[error]Node run failed with exit code 1

@jbj
Copy link
Copy Markdown
Contributor

jbj commented Dec 13, 2019

That got fixed in #2527, but the check is still failing.

@RasmusWL
Copy link
Copy Markdown
Member

yes, since it will use the version of the yml file that is on the PR, it will all work in the GloriusFuture™.

@tausbn tausbn deleted the automatically-label-pull-requests branch February 12, 2021 18:02
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.

6 participants