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

Drop access-control from biblio.json, now handled in w3c.json #830

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

tidoust
Copy link
Collaborator

@tidoust tidoust commented Oct 21, 2024

access-control was marked as an alias of fetch. That's not incorrect, but history-wise, access-control is more an alias of cors, which got superseded by fetch. That history wasn't properly recorded in the W3C database. It now is, so W3C update script is currently stuck because it wants to add an access-control entry that targets cors, duplicating the entry in biblio.json. This update drops the entry from biblio.json, so that updates may resume.

(The superseding relationship between cors and fetch is not properly captured yet but that's orthogonal to the problem at hand, see #829).

`access-control` was marked as an alias of `fetch`. That's not incorrect, but
history-wise, `access-control` is more an alias of `cors`, which got superseded
by `fetch`. That history wasn't properly recorded in the W3C database. It now
is, so W3C update script is currently stuck because it wants to add an
`access-control` entry that targets `cors`, duplicating the entry in
`biblio.json`. This update drops the entry from `biblio.json`, so that updates
may resume.

(The superseding relationship between `cors` and `fetch` is not properly
captured yet but that's orthogonal to the problem at hand, see tobie#829).
@tidoust
Copy link
Collaborator Author

tidoust commented Oct 21, 2024

... And the update also adds the entry that the W3C update script would add to w3c.json to avoid a temporary hiccup where the entry would have disappeared.

@tidoust tidoust requested a review from deniak October 21, 2024 09:41
@tidoust tidoust merged commit 3061d1f into tobie:main Oct 21, 2024
1 check passed
@tidoust tidoust deleted the access-control branch October 21, 2024 11:40
dr-shorthair pushed a commit to dr-shorthair/specref that referenced this pull request Nov 29, 2024
…e#830)

* Drop `access-control` from biblio.json, now handled in w3c.json

`access-control` was marked as an alias of `fetch`. That's not incorrect, but
history-wise, `access-control` is more an alias of `cors`, which got superseded
by `fetch`. That history wasn't properly recorded in the W3C database. It now
is, so W3C update script is currently stuck because it wants to add an
`access-control` entry that targets `cors`, duplicating the entry in
`biblio.json`. This update drops the entry from `biblio.json`, so that updates
may resume.

(The superseding relationship between `cors` and `fetch` is not properly
captured yet but that's orthogonal to the problem at hand, see tobie#829).
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