Skip to content

Fix RSS feed #1505

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 1 commit into from
Mar 8, 2025
Merged

Fix RSS feed #1505

merged 1 commit into from
Mar 8, 2025

Conversation

senekor
Copy link
Contributor

@senekor senekor commented Mar 8, 2025

This was broken in the transition from Hanlebars to Tera. It was excluded from the snapshot tests, because it contains a non-deterministic timestamp.

reported here: #1502 (comment)

This was broken in the transition from Hanlebars to Tera. It
was excluded from the snapshot tests, because it contains a
non-deterministic timestamp.
@senekor senekor requested a review from Manishearth March 8, 2025 07:01
@senekor
Copy link
Contributor Author

senekor commented Mar 8, 2025

cargo-insta has Redactions and Filters to ignore non-deterministic parts on a fine-grained level, I should've used that instead of excluding the whole file.

@Manishearth Manishearth merged commit 30d835d into master Mar 8, 2025
3 checks passed
@Manishearth Manishearth deleted the senekor/srtpvxvmlopo branch March 8, 2025 07:12
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