Skip to content

Add thread locker workflow#5909

Merged
sigmavirus24 merged 1 commit intomainfrom
lock-threads
Aug 25, 2021
Merged

Add thread locker workflow#5909
sigmavirus24 merged 1 commit intomainfrom
lock-threads

Conversation

@sigmavirus24
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

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

Just a minor fix needed and then I think this looks good. Thanks, @sigmavirus24!

@@ -0,0 +1,15 @@
me: 'Lock Threads'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks like this got clipped during copy. me->name

action:
runs-on: ubuntu-latest
steps:
- uses: dessant/lock-threads@v2
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Actually one more thought while we're here. It looks like some other projects have shortened the window from 365 days to 90. That may be more reasonable to prevent unrelated pings resetting such a long timer.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oh, I missed this. Hold on. #5911

Copy link
Copy Markdown
Member

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

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

🚀

@sigmavirus24 sigmavirus24 merged commit 7adb868 into main Aug 25, 2021
@sigmavirus24 sigmavirus24 deleted the lock-threads branch August 25, 2021 18:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants