Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Improve contribution guidelines#13902

Merged
paritytech-processbot[bot] merged 1 commit into
masterfrom
bkchr-improve-contributions
Apr 28, 2023
Merged

Improve contribution guidelines#13902
paritytech-processbot[bot] merged 1 commit into
masterfrom
bkchr-improve-contributions

Conversation

@bkchr
Copy link
Copy Markdown
Member

@bkchr bkchr commented Apr 13, 2023

No description provided.

@bkchr bkchr requested a review from the-right-joyce April 13, 2023 12:00
@bkchr bkchr added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Apr 13, 2023
@bkchr
Copy link
Copy Markdown
Member Author

bkchr commented Apr 13, 2023

@crystalin I would be really happy that if you have more ideas on how to improve your downstream experience, that you mention it here.

Copy link
Copy Markdown
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

What about a PR template that helps with this? I know some people in DevRel are thinking about this.

@bkchr
Copy link
Copy Markdown
Member Author

bkchr commented Apr 13, 2023

What about a PR template that helps with this? I know some people in DevRel are thinking about this.

We clearly need to improve the current PR template. IMO it is too much and people probably just skip it.

@ggwpez
Copy link
Copy Markdown
Member

ggwpez commented Apr 26, 2023

@crystalin I would be really happy that if you have more ideas on how to improve your downstream experience, that you mention it here.

ping

Otherwise merging soon.

@ggwpez
Copy link
Copy Markdown
Member

ggwpez commented Apr 28, 2023

bot merge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants