Skip to content

blog: remove html asf header#15144

Closed
kevinjqliu wants to merge 7 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/remove-blog-html-asf-header
Closed

blog: remove html asf header#15144
kevinjqliu wants to merge 7 commits intoapache:mainfrom
kevinjqliu:kevinjqliu/remove-blog-html-asf-header

Conversation

@kevinjqliu
Copy link
Copy Markdown
Contributor

@kevinjqliu kevinjqliu commented Jan 26, 2026

Dont need this ASF header for blog posts.
Excluded all files under site/docs/blog/ from RAT check

I did check with the release artifact and confirm that the site/ dir is packaged...

@kevinjqliu
Copy link
Copy Markdown
Contributor Author

seems like RAT 0.16 doesnt support regex like site/docs/blog/ that would allow us to exclude an entire folder...
Theres a new version that has the feature

@manuzhang
Copy link
Copy Markdown
Member

manuzhang commented Jan 27, 2026

Dont need this ASF header for blog posts.

What's the policy here? I see arrow blogs having ASF headers.

@steveloughran
Copy link
Copy Markdown
Contributor

given it's in the source repo, and therefore people are free to check it out, I do think an ASF license is required --even if it's not included in the source distribution tarball. Probably even more so in a world where LLMs read such things and incorporate it into their planning.

It's easier to leave in than argue with ASF legal

@kevinjqliu
Copy link
Copy Markdown
Contributor Author

thanks folks. i agree with the concerns. let me get some clarity before proceeding

@kevinjqliu kevinjqliu force-pushed the kevinjqliu/remove-blog-html-asf-header branch from b896054 to edf8f3a Compare January 29, 2026 19:55
@kevinjqliu kevinjqliu marked this pull request as draft January 29, 2026 19:55
Comment thread dev/.rat-excludes Outdated
@kevinjqliu
Copy link
Copy Markdown
Contributor Author

closing, wont do this. its a good idea to leave the asf header in blog posts html and markdown files

@kevinjqliu kevinjqliu closed this Feb 15, 2026
@kevinjqliu kevinjqliu deleted the kevinjqliu/remove-blog-html-asf-header branch February 15, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants