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

Bug actions #37

Merged
merged 2 commits into from
Feb 28, 2024
Merged

Bug actions #37

merged 2 commits into from
Feb 28, 2024

Conversation

stianst
Copy link
Collaborator

@stianst stianst commented Feb 22, 2024

No description provided.

@stianst stianst force-pushed the bug-actions branch 2 times, most recently from 317faf4 to aa9f5e2 Compare February 22, 2024 09:26
@stianst stianst force-pushed the bug-actions branch 2 times, most recently from dd27e44 to 57ae729 Compare February 22, 2024 09:31
Signed-off-by: stianst <stianst@gmail.com>
@stianst stianst marked this pull request as ready for review February 22, 2024 14:46
@stianst stianst requested a review from ssilvert February 26, 2024 07:53
Signed-off-by: stianst <stianst@gmail.com>
contribution to fix the issue.

expired=\
Due to lack of updates in the last {0} {1} this issue will be automatically closed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we somehow reflect the case when someone else comes across that issue in some time?

Suggested change
Due to lack of updates in the last {0} {1} this issue will be automatically closed.
Due to lack of updates in the last {0} {1} this issue will be automatically closed. If you believe this issue has a higher priority and should be fixed, feel free to open a new issue. Please note we do not monitor closed issues.

Comment on lines +13 to +14
to evaluate the issue. Please verify with the [nightly build](https://github.com/keycloak/keycloak/releases/tag/nightly)\
or the [latest release](https://www.keycloak.org/downloads).\n\
Copy link
Contributor

Choose a reason for hiding this comment

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

I know this corresponds with the current bug template but it feels to be a bit odd to allow reports against latest release if we have nightlies. Do we really want to waste time triaging issues that are fixed or no longer present on nightly but affect latest release?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We should not require folks to use the nightly, and they should be able to report issues against any supported release.

Comment on lines +27 to +28
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue.
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue. If there are no updates to this issue, it'll be closed in 180 days.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Messages shouldn't include the clean-up part. We may decide to change the number of days, and we may close issues at any given time if there's no updates.

Comment on lines +33 to +34
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue.
If you are affected by this issue, upvote it by adding a :thumbsup: to the description. We would also welcome a \
contribution to fix the issue. If there are no updates to this issue, it'll be closed in 90 days.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Messages shouldn't include the clean-up part. We may decide to change the number of days, and we may close issues at any given time if there's no updates.

@stianst stianst merged commit 380b28c into keycloak:main Feb 28, 2024
2 checks passed
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.

3 participants