Skip to content

Conversation

@noopurintel
Copy link
Collaborator

@noopurintel noopurintel commented Feb 18, 2025

This PR aims to skip the PR pipeline run for changes done to specific files which won't impact the functionality anyways.

The list includes:

  1. **.md (includes CODE_OF_CONDUCT.md, CONTRIBUTING.md, GOVERNANCE.md, README.md, ROADMAP.md, SECURITY.md and TSC.md)
  2. docs/**
  3. .dockerignore
  4. .gitattributes
  5. .gitignore
  6. CITATION
  7. CODEOWNERS

IMP: If any other file is included outside this list - either in the same commit or a subsequent one, the pipeline would run as is.

In addition, windows.yml has been modified NOT to run for PRs in draft stage.

@noopurintel noopurintel changed the title Skip pipeline runs for doc changes Do not review: Skip pipeline runs for doc changes Feb 18, 2025
@noopurintel noopurintel marked this pull request as draft February 18, 2025 05:15
@noopurintel noopurintel marked this pull request as ready for review February 18, 2025 05:18
@noopurintel noopurintel changed the title Do not review: Skip pipeline runs for doc changes Skip pipeline runs for specific file changes Feb 18, 2025
@MasterSkepticista MasterSkepticista merged commit a7b67cd into securefederatedai:develop Feb 18, 2025
33 checks passed
yuliasherman pushed a commit to yuliasherman/openfl that referenced this pull request Mar 18, 2025
* Skip pipeline runs for doc changes

Signed-off-by: noopur <[email protected]>

* Skip pipeline runs for doc changes

Signed-off-by: noopur <[email protected]>

* Included more files

Signed-off-by: noopur <[email protected]>

* Included more files

Signed-off-by: noopur <[email protected]>

* Corrected the step for windows workflow

Signed-off-by: noopur <[email protected]>

* Remove scheduler for windows wf

Signed-off-by: noopur <[email protected]>

---------

Signed-off-by: noopur <[email protected]>
@noopurintel noopurintel deleted the noopur/skip-pipeline-for-docs branch March 26, 2025 03:51
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.

6 participants