Skip to content

[fix][CI] Don't run pulsar-ci-flaky workflow for PRs which contain only document changes - #17906

Merged
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-dont-run-pulsar-ci-flaky-for-docs-only-prs
Sep 30, 2022
Merged

[fix][CI] Don't run pulsar-ci-flaky workflow for PRs which contain only document changes#17906
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-dont-run-pulsar-ci-flaky-for-docs-only-prs

Conversation

@lhotari

@lhotari lhotari commented Sep 30, 2022

Copy link
Copy Markdown
Member

Motivation

There's a typo in pulsar-ci-flaky workflow definition which results in the workflow to get run for document only changes.
This was a typo in #17881 at https://github.com/apache/pulsar/pull/17881/files#diff-3559f910180dd3552d2e3708c794b1826ef5ac328ff28657a4fabd1201b61fddR74

Modifications

  • fix typo

Documentation

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why)

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Sep 30, 2022

@nicoloboschi nicoloboschi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

good catch

we should set CI jobs to test our CI jobs 😵‍💫

@lhotari
lhotari merged commit c0b3039 into apache:master Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants