Skip to content

Conversation

@yaooqinn
Copy link
Member

What changes were proposed in this pull request?

This PR adds the auto-generated content docs/_generated/ to .gitignore

Why are the changes needed?

vcs improvement

Does this PR introduce any user-facing change?

no

How was this patch tested?

existing tests

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

no

@github-actions github-actions bot added the INFRA label Apr 23, 2024
@yaooqinn yaooqinn changed the title [MINOR][DOCS] [MINOR][DOCS] Add docs/_generated/ to .gitignore [MINOR][DOCS] Add docs/_generated/ to .gitignore Apr 23, 2024
@dongjoon-hyun
Copy link
Member

Merged to master.

@yaooqinn yaooqinn deleted the minor branch April 24, 2024 03:51
@yaooqinn
Copy link
Member Author

Thank you @dongjoon-hyun @zhengruifeng

@nchammas
Copy link
Contributor

Whoops, I believe this is the fault of #44971. I have the ignore in my local checkout but I didn't add it to the PR. Thanks for patching that up.

@yaooqinn
Copy link
Member Author

yaooqinn commented Apr 24, 2024

Thank you for the double check @nchammas

yaooqinn added a commit that referenced this pull request Jul 11, 2024
### What changes were proposed in this pull request?

This PR adds the auto-generated content `docs/_generated/` to .gitignore

### Why are the changes needed?

vcs improvement

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

no

### How was this patch tested?

existing tests

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

no

Closes #46178 from yaooqinn/minor.

Authored-by: Kent Yao <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
@yaooqinn
Copy link
Member Author

cherry-picked this to branch-3.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants