ci: move node-validation to seprate file

[Dunqing]

Description

pnpm format:ci will format Markdown files. but the ci action will ignore markdown changes. So I think we should do a lint check in the new action.

Test Plan

---

[Dunqing]

• ci: move node-validation to seprate file #584 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on Graphite

[netlify]

✅ Deploy Preview for rolldown-rs canceled.

Name	Link
🔨 Latest commit	929cf12
🔍 Latest deploy log	https://app.netlify.com/sites/rolldown-rs/deploys/65f44c5b0101930008df0e0d

[hyf0]

I get you. But the better solution would be removing paths-ignore: in CI.yaml. Due to the "Status check" of github is static and running these actions are required. We need to bare with useless action running for some PRs.

[hyf0]

I get you. But the better solution would be removing paths-ignore: in CI.yaml. Due to the "Status check" of github is static and running these actions are required. We need to bare with useless action running for some PRs.

Or should we say the path ignore is not smart enough. It should treat skipped actions as completed successfully rather than skipped.

[Dunqing]

I get you. But the better solution would be removing paths-ignore: in CI.yaml. Due to the "Status check" of github is static and running these actions are required. We need to bare with useless action running for some PRs.

CI action has many jobs, As far as I know only node-validation needs to remove path_ignores. but this causes all other jobs to be executed, so I moved it to a new action to avoid useless CI check

[hyf0]

I get you. But the better solution would be removing paths-ignore: in CI.yaml. Due to the "Status check" of github is static and running these actions are required. We need to bare with useless action running for some PRs.

CI action has many jobs, As far as I know only node-validation needs to remove path_ignores. but this causes all other jobs to be executed, so I moved it to a new action to avoid useless CI check

As I was saying, it's useless and we have to bare with it. You will notice that currently your PR doesn't not run cargo test action and it's required to merge this PR. The problem is that github can't set what actions are required dynamically, so we need to set needed actions required. That's where we hit the "path ignore" problem.

Skipped actions by "path ignore" won't pass the checks of github.

[hyf0]

Let's merge this one first. I will refactor them using https://github.com/dorny/paths-filter later.