Skip to content

feat: Add robots.txt to site - #24333

Merged
Jefffrey merged 8 commits into
apache:mainfrom
imtherealnaska:robot-23258
Aug 16, 2026
Merged

Jefffrey merged 8 commits into
apache:mainfrom
imtherealnaska:robot-23258

Conversation

@imtherealnaska

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

What changes are included in this PR?

There are 2 PR's for this issue. This one contains the changes for robots.txt

  • added the robots.txt , added license header and allow rules.

Are these changes tested?

Yes. Tested by building the docs and verifying the files getting generated in the build/

Are there any user-facing changes?

No.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 13, 2026
@imtherealnaska
imtherealnaska marked this pull request as ready for review August 13, 2026 17:03
@codecov-commenter

codecov-commenter commented Aug 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.23%. Comparing base (110a90a) to head (9c21847).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #24333      +/-   ##
==========================================
- Coverage   81.23%   81.23%   -0.01%     
==========================================
  Files        1111     1111              
  Lines      390208   390208              
  Branches   390208   390208              
==========================================
- Hits       316991   316983       -8     
- Misses      54592    54594       +2     
- Partials    18625    18631       +6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Jefffrey Jefffrey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

dependent on #24332 since i think we want to include the sitemap as part of the robots.txt

Comment thread docs/source/robots.txt Outdated
@@ -0,0 +1,19 @@
# Licensed to the Apache Software Foundation (ASF) under one

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not sure if we're supposed to have this license txt in a robots.txt?

@imtherealnaska imtherealnaska Aug 14, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I added this because , '#' will be ignored by crawlers also I had to add the robots.txt file name to rat_exclude_ files.txt. Im happy to remove it , fine either way .

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks. I have added a commit removing license and following what llms.txt followed.

@github-actions github-actions Bot added the development-process Related to development process of DataFusion label Aug 16, 2026
@Jefffrey Jefffrey changed the title feat: Add robots.txt and License file feat: Add robots.txt to site Aug 16, 2026
@Jefffrey
Jefffrey added this pull request to the merge queue Aug 16, 2026
Merged via the queue into apache:main with commit bbf739a Aug 16, 2026
39 checks passed
@Jefffrey

Copy link
Copy Markdown
Contributor

live now: https://datafusion.apache.org/robots.txt

thanks @imtherealnaska

@imtherealnaska

Copy link
Copy Markdown
Contributor Author

wooohooo. first commit . @Jefffrey Thanks for the help.

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

Labels

development-process Related to development process of DataFusion documentation Improvements or additions to documentation v56.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve search engine / agent discoverability for DataFusion website

4 participants