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 pull request guidelines and template. #1767

Merged

Conversation

aucampia
Copy link
Member

@aucampia aucampia commented Mar 19, 2022

Summary of changes

This PR adds pull request guidelines to the developers guide, it also adds a pull
request template that makes reference to these guidelines with a checklist to remind
people of the most essential preconditions for merging a PR and some less essential
but useful things.

This is mainly to help people understand how we process PRs and what
they can do to make it more likely for their PRs to be processed quickly.

Checklist

  • Checked that there aren't other open pull requests for
    the same change.
  • Checked that all tests pass.
  • Considered granting push permissions to the PR branch,
    so maintainers can fix minor issues and keep your PR up to date.

@aucampia aucampia force-pushed the iwana-20220315T2240-pr_guidelines branch 2 times, most recently from f38f7d1 to a415950 Compare March 19, 2022 14:59
This is mainly to help people understand how we process PRs and what
they can do to make it more likely for their PRs to be processed qickly.
@aucampia aucampia force-pushed the iwana-20220315T2240-pr_guidelines branch from a415950 to ead90ea Compare March 19, 2022 15:03
@aucampia aucampia marked this pull request as ready for review March 19, 2022 15:04
Copy link
Member

@white-gecko white-gecko left a comment

Choose a reason for hiding this comment

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

Great work, thank you! Can be merged.

Copy link
Member

@nicholascar nicholascar left a comment

Choose a reason for hiding this comment

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

Thanks @aucampia

@nicholascar nicholascar merged commit 7df1841 into RDFLib:master Mar 20, 2022
@aucampia aucampia deleted the iwana-20220315T2240-pr_guidelines branch April 9, 2022 14:09
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.

None yet

3 participants