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 write access request form #7

Merged
merged 10 commits into from
Jun 29, 2023
Merged

Conversation

timtebeek
Copy link
Contributor

What's changed?

Added write access request form, which will then show up in each OpenRewrite repository, when people go to open a new issue.

What's your motivation?

We want to document the way people can get involved with the project, especially for authors of library and frameworks we already cover.

Anything in particular you'd like reviewers to focus on?

  • Is it clear enough that we won't as easily grant write access to just anyone,
    but specifically to people already involved with the library or framework covered in each module?
  • Is it sufficiently clear from the comment at the bottom that we do expect some form of commitment to the project?
  • Any other fields we might like to ask about?

Have you considered any alternatives or workarounds?

We could also only have this form in a dedicated repository for access requests, but that makes it harder to find when you come in through for instance rewrite-hibernate.

Any additional context

openrewrite/rewrite-docs#188

@timtebeek timtebeek added the documentation Improvements or additions to documentation label Jun 29, 2023
@timtebeek timtebeek self-assigned this Jun 29, 2023
@timtebeek
Copy link
Contributor Author

Copy link

@rpau rpau left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@mike-solomon mike-solomon left a comment

Choose a reason for hiding this comment

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

Two minor suggestions but other than that looks good to me. Thanks for doing this!

.github/ISSUE_TEMPLATE/write_access_request.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/write_access_request.yml Outdated Show resolved Hide resolved
@timtebeek timtebeek merged commit 51e52f0 into main Jun 29, 2023
@timtebeek timtebeek deleted the tim/write_access_request_form branch June 29, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants