Skip to content

[SPARK-58009][INFRA] Add build/test irrelevant files to dev_tools module#57090

Closed
gaogaotiantian wants to merge 1 commit into
apache:masterfrom
gaogaotiantian:extra-files-to-ignore
Closed

[SPARK-58009][INFRA] Add build/test irrelevant files to dev_tools module#57090
gaogaotiantian wants to merge 1 commit into
apache:masterfrom
gaogaotiantian:extra-files-to-ignore

Conversation

@gaogaotiantian

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add some build/test irrelevant files to dev_tools module.

Why are the changes needed?

So changes to these files won't trigger a full build in CI.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI.

Was this patch authored or co-authored using generative AI tooling?

No.

@dongjoon-hyun dongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM. Thank you for proposing this dev_tools module and maintaining like this, @gaogaotiantian .

@gaogaotiantian

Copy link
Copy Markdown
Contributor Author

Thank you @dongjoon-hyun !

gaogaotiantian added a commit that referenced this pull request Jul 13, 2026
…odule

### What changes were proposed in this pull request?

Add some build/test irrelevant files to `dev_tools` module.

### Why are the changes needed?

So changes to these files won't trigger a full build in CI.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #57090 from gaogaotiantian/extra-files-to-ignore.

Authored-by: Tian Gao <gaogaotiantian@hotmail.com>
Signed-off-by: Tian Gao <gaogaotiantian@hotmail.com>
(cherry picked from commit 9b7d3dc)
Signed-off-by: Tian Gao <gaogaotiantian@hotmail.com>
@gaogaotiantian

Copy link
Copy Markdown
Contributor Author

Merge Summary:

Posted by merge_spark_pr.py

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