Skip to content

Conversation

@eyJhb
Copy link
Contributor

@eyJhb eyJhb commented Jun 6, 2025

Stumbled upon this typo when doing #2643 , and thought I might as well change it.
Discovered that the error is no longer used, after #2398 was merged.

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

@ghost
Copy link

ghost commented Jun 6, 2025

Pull Request Revisions

RevisionDescription
r2
Removed robots.txt endpoint handlerDeleted the RobotsHandler and its route registration from the application, removing the previously added /robots.txt endpoint
r1
Added robots.txt endpoint handlerImplemented a new /robots.txt endpoint handler to prevent web crawlers from accessing the server

✅ AI review completed for r2
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at [email protected].

@eyJhb eyJhb force-pushed the hscontrol-handler-typo branch from 44188bf to f23a12e Compare June 6, 2025 09:45
@eyJhb
Copy link
Contributor Author

eyJhb commented Jun 8, 2025

I don't think I can do anything about the linting error, otherwise, please correct me if there is anything more I need to do.

image

@kradalby kradalby merged commit efc6974 into juanfont:main Jul 4, 2025
141 of 150 checks passed
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