Skip to content

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Mar 24, 2025

This creates a new way to curate the results of a crawl: a freeze patch allows to freeze the results of a crawl to a specific commit ID in Webref. This is meant for specs that are temporarily broken beyond repair.

First such patch is for the filter-effects-1 spec.

Each freeze patch is implemented as a JSON file named after the spec's shortname and that contains the commit ID and a pending key that links to the issue that tracks the problem in some GitHub repository. The clean-patches script proposes to drop the patch when the issue gets closed.

This creates a new way to curate the results of a crawl: a freeze patch
allows to freeze the results of a crawl to a specific commit ID in Webref. This
is meant for specs that are temporarily broken beyond repair.

First such patch is for the filter-effects-1 spec.

Each freeze patch is implemented as a JSON file named after the spec's
shortname and that contains the commit ID and a `pending` key that links to the
issue that tracks the problem in some GitHub repository. The `clean-patches`
script proposes to drop the patch when the issue gets closed.
@tidoust tidoust requested a review from dontcallmedom March 24, 2025 11:56
Copy link
Member

@dontcallmedom dontcallmedom left a comment

Choose a reason for hiding this comment

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

splendid, thanks! a few minor suggestions for your consideration

tidoust added 3 commits March 24, 2025 14:58
This generalizes the logic that parses property values of a crawn index to
gather extract files, and applies it throughout. This actually fixes the
`removeFromCuration` logic that was broken for CDDL.
@tidoust tidoust merged commit 2f15bb4 into main Mar 24, 2025
1 check passed
@tidoust tidoust deleted the freeze-patches branch March 24, 2025 14:45
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