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

create .github/pr-custom-review.yml #5253

Merged
merged 1 commit into from
Apr 7, 2022
Merged

Conversation

joao-paulo-parity
Copy link
Contributor

⚠️ The inclusion of pr-custom-review.yml does not incur immediate application of its rules. This file will start to be used only after the GitHub workflow is added, as demonstrated by https://github.com/paritytech/substrate/pull/10951/files.


This PR implements the requirements discussed in paritytech/pr-custom-review#67. Feel free to comment there for any disagreements.

The missing requirement is implemented as a built-in check, therefore it should not be included here.

Creates .github/pr-custom-review.yml

@joao-paulo-parity joao-paulo-parity requested review from a team and chevdor as code owners April 5, 2022 13:50
@joao-paulo-parity joao-paulo-parity changed the title create pr-custom-review.yml create .github/pr-custom-review.yml Apr 5, 2022
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Apr 5, 2022
@joao-paulo-parity joao-paulo-parity added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. and removed A0-please_review Pull request needs code review. labels Apr 5, 2022
@sergejparity sergejparity requested review from a team April 6, 2022 13:14
@TriplEight TriplEight merged commit fcff49a into master Apr 7, 2022
@TriplEight TriplEight deleted the jp/pr-custom-review-config branch April 7, 2022 14:23
- name: Runtime files
check_type: changed_files
condition: ^runtime/(kusama|polkadot)/src/[^/]+\.rs$
all_distinct:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bkchr to make the review work like you suggest in paritytech/pr-custom-review#81 (comment) for this rule, change this line

Suggested change
all_distinct:
all:

It would not affect 🔒 lines however

ordian added a commit that referenced this pull request Apr 13, 2022
* master: (62 commits)
  Bump tracing from 0.1.32 to 0.1.33 (#5299)
  Add staging runtime api (#5048)
  CI: rename ambiguous jobs (#5313)
  Prepare for rust 1.60 (#5282)
  Bump proc-macro2 from 1.0.36 to 1.0.37 (#5265)
  Fixes the dead lock when any of the channels get at capacity. (#5297)
  Bump syn from 1.0.90 to 1.0.91 (#5273)
  create .github/workflows/pr-custom-review.yml (#5280)
  [ci] fix pallet benchmark script (#5247) (#5292)
  bump spec_version to 9190 (#5291)
  bump version to 0.9.19 (#5290)
  Session is actually ancient not unknown. (#5286)
  [ci] point benchmark script to sub-commands (#5247) (#5289)
  Remove Travis CI (#5287)
  Improve error handling in approval voting block import (#5283)
  fix .github/pr-custom-review.yml (#5279)
  Co #11164: Sub-commands for `benchmark` (#5247)
  Bump clap from 3.1.6 to 3.1.8 (#5240)
  Custom Slots Migration for Fund Index Change (#5227)
  create pr-custom-review.yml (#5253)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants