Skip to content

Conversation

@GOODBOY008
Copy link
Member

Description

This PR updates the CI workflow to ignore specific file paths for pull_request and push events. This prevents the CI pipeline from running unnecessarily when changes are limited to documentation or other non-code files.

Motivation

Currently, CI runs for all changes, including documentation updates. For example, in #710, the CI was triggered for purely documentation changes. This PR adds paths-ignore to exclude files like website/**, **.md, LICENSE, and NOTICE to save resources and speed up the development process.

Key Changes

  • Modified .github/workflows/ci.yml to add paths-ignore configuration for pull_request and push events.

@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for fesod ready!

Name Link
🔨 Latest commit e73f120
🔍 Latest deploy log https://app.netlify.com/projects/fesod/deploys/69259d9a0a08f80008035aa3
😎 Deploy Preview https://deploy-preview-711--fesod.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@delei
Copy link
Member

delei commented Nov 24, 2025

Is this similar to PR #705 ?

@GOODBOY008 GOODBOY008 closed this Nov 24, 2025
@GOODBOY008
Copy link
Member Author

Is this similar to PR #705 ?

@delei I think my impl is better. PTAL

@GOODBOY008 GOODBOY008 reopened this Nov 24, 2025
@delei
Copy link
Member

delei commented Nov 24, 2025

Hi, @GOODBOY008
It’s fine.

😄 Might you be willing to leave a comment in #705 to guide new contributors?

@psxjoy psxjoy requested a review from delei November 25, 2025 12:14
Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@delei delei merged commit ba9d29e into apache:main Nov 25, 2025
9 checks passed
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.

3 participants