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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(pinner): Create issues-pinner.yml workflow #7056

Merged
merged 6 commits into from
Sep 14, 2022

Conversation

davorpa
Copy link
Member

@davorpa davorpa commented Sep 3, 2022

What does this PR do?

For resources

Description

A workflow to monitor pin/unpin issue event types and add/remove state labels accordingly

Needs declare a new label: 馃搶 pinned :pushpin: pinned

Resolves #7055

Workflow runs:
- Tested with davorpa#16 issue
- https://github.com/davorpa/free-programming-books/actions/runs/2983767547
- https://github.com/davorpa/free-programming-books/actions/runs/2983768434
- https://github.com/davorpa/free-programming-books/actions/runs/2983769061
- https://github.com/davorpa/free-programming-books/actions/runs/2983769167

Checklist:

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

To monitor pin/unpin issue events and add/remove state labels accordingly
@davorpa davorpa added 馃憖 Needs Review Is this really a good resource? Reviews requested. 馃 automation Automated tasks done by workflows or bots labels Sep 3, 2022
@davorpa davorpa self-assigned this Sep 3, 2022
@davorpa davorpa marked this pull request as ready for review September 3, 2022 08:53
@LuigiImVector LuigiImVector added the New Feature New feature / enhancement / translation... label Sep 3, 2022
Co-authored-by: ImVector <59611597+LuigiImVector@users.noreply.github.com>
@LuigiImVector LuigiImVector removed the 馃憖 Needs Review Is this really a good resource? Reviews requested. label Sep 4, 2022
@LuigiImVector LuigiImVector added the 馃殌 ready to merge LGTM. Waiting for final approval label Sep 4, 2022
@eshellman
Copy link
Collaborator

do you know of another repo where this is done?

@eshellman
Copy link
Collaborator

any comments, @mohitsaxenaknoldus ?

@davorpa
Copy link
Member Author

davorpa commented Sep 5, 2022

do you know of another repo where this is done?

Not directly. But workflow is easy to newbies. I apply some merged knowledges:

@davorpa davorpa removed the 馃殌 ready to merge LGTM. Waiting for final approval label Sep 6, 2022
@davorpa davorpa marked this pull request as draft September 6, 2022 10:21
@davorpa davorpa marked this pull request as ready for review September 6, 2022 10:49
@davorpa davorpa mentioned this pull request Sep 7, 2022
2 tasks
@davorpa davorpa marked this pull request as draft September 13, 2022 22:16
@davorpa
Copy link
Member Author

davorpa commented Sep 13, 2022

@davorpa davorpa marked this pull request as ready for review September 13, 2022 22:21
@eshellman
Copy link
Collaborator

please add a header comment in the code that describes the purpose of the action and exactly what it does.

@davorpa
Copy link
Member Author

davorpa commented Sep 14, 2022

please add a header comment in the code that describes the purpose of the action and exactly what it does.

This workflow adds a label to an issue when is pinned and removes it when unpinned.

The label is enhanced with stale.yml workflow. Pinned issues never stales due to the label is declared as exemption

@eshellman
Copy link
Collaborator

This workflow adds a label to an issue when is pinned and removes it when unpinned.

The label is enhanced with stale.yml workflow. Pinned issues never stales due to the label is declared as exemption

that helps. now add it to the file

@davorpa
Copy link
Member Author

davorpa commented Sep 14, 2022

that helps. now add it to the file

Done 01663e6

@LuigiImVector
Copy link
Member

Nice work

@LuigiImVector LuigiImVector added the 馃殌 ready to merge LGTM. Waiting for final approval label Sep 14, 2022
@eshellman eshellman merged commit d16c967 into EbookFoundation:main Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
馃 automation Automated tasks done by workflows or bots New Feature New feature / enhancement / translation... 馃殌 ready to merge LGTM. Waiting for final approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Create a workflow that label/unlabel issues on pin/unpin
3 participants