Skip to content

CC-NEWS support and validation for crawl reference #11

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

Merged
merged 5 commits into from
Mar 7, 2025
Merged

Conversation

pjox
Copy link
Member

@pjox pjox commented Mar 6, 2025

Description

This PR introduces support for CC-NEWS and adds validations for the crawl or snapshot references. This PR also updates some libraries and bumps the rust edition to 2024 and the latest 1.85 version.

Breaking Changes

No Breaking changes

Notes & open questions

The error messages can still be fine-tuned a little bit.

pjox added 3 commits March 6, 2025 13:53
…sn't exist and cc-downloader downloaded the body of the response

Now this action will produce an error
…the 4XX error message when downloading paths, adds validation to the cli input for the crawl reference
@pjox pjox added the enhancement New feature or request label Mar 6, 2025
@pjox pjox requested a review from thunderpoot March 6, 2025 14:33
@pjox pjox self-assigned this Mar 6, 2025
… automatically fix the casing of the crawl reference
Copy link
Member

@thunderpoot thunderpoot left a comment

Choose a reason for hiding this comment

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

That's brilliant, thanks for addressing the review comments, and I hereby give approval even though you didn't ask for it 😂

@pjox pjox merged commit bc24e45 into dev Mar 7, 2025
1 check passed
@pjox pjox deleted the pjox/404-fix branch March 7, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants