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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a ruletype that checks the issue tracker status #254

Closed
wants to merge 2 commits into from

Conversation

rdimitrov
Copy link
Member

The following ruletype checks the issue tracker status (whether it is enabled or disabled).

It should satisfy OSPOS-DO-05

@rdimitrov rdimitrov requested a review from a team as a code owner December 20, 2024 08:32
@rdimitrov rdimitrov self-assigned this Dec 20, 2024
Signed-off-by: Radoslav Dimitrov <[email protected]>
type: boolean
description: "Whether the repository should have issues enabled (true) or disabled (false)."
required:
- has_issues
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if we want to have an explicit rule that verifies that the issue tracker is indeed enabled. This would make the rule simpler since it would not have a toggle, and we could tailor the guidance to that case. wdyt?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sounds good 👍 So splitting this to 2 separate rules, i.e. issues_enabled and issues_disabled?

Copy link
Contributor

Choose a reason for hiding this comment

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

love it

Copy link
Member Author

Choose a reason for hiding this comment

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

on it 🚀

@rdimitrov
Copy link
Member Author

Closing this in favour of #255 and #256

@rdimitrov rdimitrov closed this Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants