Skip to content

Conversation

@korikuzma
Copy link
Member

I'm working on adding GH Organizational Rulesets and noticed PR priority label job was missing from this repo.

@korikuzma korikuzma self-assigned this Dec 23, 2025
@korikuzma korikuzma requested a review from a team December 23, 2025 18:52
@korikuzma korikuzma added ci Continuous integration changes priority:high High priority priority:low Low priority and removed priority:high High priority labels Dec 23, 2025
@korikuzma
Copy link
Member Author

@GenomicMedLab/anyvlm-maintainers could this get a review from someone?

Copy link
Member

@jsstevenson jsstevenson left a comment

Choose a reason for hiding this comment

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

👍 but I think this should have buy-in from external collaborators too

@korikuzma
Copy link
Member Author

👍 but I think this should have buy-in from external collaborators too

This is our lab policy:

We require a priority label to help reviewers determine when they should complete the review.
priority:high – review within 1 day
priority:medium – review within 2-3 days
priority:low – review within 1 week
If you do not get a review by a developer within the time frame, feel free to tag them in a comment or message them.

We're going to add a priority:critical.

@theferrit32 @toneillbroad @quinnwai Thoughts? We can be flexible with review days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous integration changes priority:low Low priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants