Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify our rationale #54

Merged
merged 1 commit into from
Nov 10, 2024
Merged

Clarify our rationale #54

merged 1 commit into from
Nov 10, 2024

Conversation

glyn
Copy link
Contributor

@glyn glyn commented Nov 9, 2024

I deleted the point about excessive load on crawled sites as any other crawler could potentially be guilty of this and I wouldn't want our scope to creep to all crawlers.

I added environmental impact, which was always in the front of my mind when contributing to this site.

Ref: #53 (comment)

I deleted the point about excessive load on
crawled sites as any other crawler could potentially
be guilty of this and I wouldn't want our scope to
creep to all crawlers.

Ref: ai-robots-txt#53 (comment)
@glyn glyn requested a review from cdransf November 9, 2024 04:50
Copy link
Member

@cdransf cdransf left a comment

Choose a reason for hiding this comment

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

Great idea! I think this is considerably clearer.

@glyn glyn merged commit 6f58ddc into ai-robots-txt:main Nov 10, 2024
@glyn glyn deleted the rationale branch November 10, 2024 00:58
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.

2 participants