Skip to content

[fix] ready-to-test is not applicible for docs only changes - #328

Merged
tisonkun merged 1 commit into
apache:mainfrom
tisonkun:ready-to-test
Dec 13, 2022
Merged

[fix] ready-to-test is not applicible for docs only changes#328
tisonkun merged 1 commit into
apache:mainfrom
tisonkun:ready-to-test

Conversation

@tisonkun

Copy link
Copy Markdown
Member

This comes from apache/pulsar#18878 (comment).

This closes #327.

Signed-off-by: tison <wander4096@gmail.com>
Co-authored-by: Anonymitaet <50226895+Anonymitaet@users.noreply.github.com>
@tisonkun
tisonkun requested a review from Anonymitaet December 13, 2022 15:07
## ready-to-test

After [PR-17693](https://github.com/apache/pulsar/pull/17693) merged, pull requests should be first tested in your own fork since the pulsar CI based on GitHub Actions has constrained resources and quota. GitHub Actions provides separate quota for pull requests that are executed in a forked repository.
After [PR-17693](https://github.com/apache/pulsar/pull/17693) merged, pull requests **except [docs only changes](https://github.com/apache/pulsar/blob/master/.github/changes-filter.yaml#L5)** should be first tested in your own fork since the pulsar CI based on GitHub Actions has constrained resources and quota. GitHub Actions provides separate quota for pull requests that are executed in a forked repository.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The contribution guide should be always up-to-date, so I hold a link to the master branch. If the master branch changes, we should update the corresponding line links.

@tisonkun

Copy link
Copy Markdown
Member Author

Merging...

If the wording can be improved further, welcome to leave a comment or send another patch.

@tisonkun
tisonkun merged commit b502bda into apache:main Dec 13, 2022
@tisonkun
tisonkun deleted the ready-to-test branch December 13, 2022 15: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.

1 participant