Skip to content

Issue Tsardom role specification #14380

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

Merged
merged 12 commits into from
Feb 11, 2022

Conversation

anatoliykmetyuk
Copy link
Contributor

@anatoliykmetyuk anatoliykmetyuk commented Jan 28, 2022

This document explicitly specifies a maintenance role that we were already having informally.

[skip ci]

- JVM backend: @sjrd (review only), @Kordyjan
- Derivation & Mirrors: @bishabosha, (@dwijnand)
- Linting (especially unused warnings) / Reporting UX : VirtusLab TBD?
- Java-compat: @Kordyjan
Copy link
Contributor Author

Choose a reason for hiding this comment

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

/cc @bishabosha @liufengyun @aherlihy @OlivierBlanvillain @abgruszecki @KacperFKorban @BarkingBad @pikinier20 @olhotak @noti0na1 @Kordyjan

CC'ing everyone who wasn't on the meeting where the list was composed – please feel free to submit your corrections if we got something wrong.

anatoliykmetyuk and others added 3 commits February 9, 2022 12:15
Co-authored-by: Paweł Marks <[email protected]>
Co-authored-by: Paweł Marks <[email protected]>
Copy link
Contributor

@nicolasstucki nicolasstucki left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

@@ -0,0 +1,75 @@
# Issue Tsar Role
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel it is time to rename the role to something meaningful such as Issue supervisor. This would help enforce the difference in how this role is taken now that it is a more serious position.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I think we need to bring it up at the next Dotty meeting to receive everyone's input about the proper name.

Other core teammates are responsible for providing information to the Issue Tsar in a timely manner when it is requested if they have that information.

## Assignment
Issue Tsar is appointed for 7 days and is responsible for what is specified in the “Responsibilities” section during those 7 days. Their assumption of the role starts from the Dotty Meeting on Monday and ends on the next Dotty Meeting on Monday.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm still wondering if it is not better to en on Friday. This way the new person has the time to fully process issues that came on the weekend. But this could be changed later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm also in favor of this, but during the meeting, the majority seemed to be in favor of the old approach. We can see how it goes and change it in the future if need be.

anatoliykmetyuk and others added 4 commits February 10, 2022 11:07
Co-authored-by: Nicolas Stucki <[email protected]>
Co-authored-by: Nicolas Stucki <[email protected]>
Co-authored-by: Nicolas Stucki <[email protected]>
Co-authored-by: Nicolas Stucki <[email protected]>
Co-authored-by: Jamie Thompson <[email protected]>
@anatoliykmetyuk anatoliykmetyuk merged commit 06891dc into scala:main Feb 11, 2022
@anatoliykmetyuk anatoliykmetyuk deleted the issue-tsar-spec branch February 11, 2022 10:22
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.

5 participants