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

Replace EnsureCSPDoesNotBlockStringCompilation section with link to upstream PR #514

Merged
merged 1 commit into from
May 14, 2024

Conversation

lukewarlow
Copy link
Member

@lukewarlow lukewarlow commented May 14, 2024

Also remove violation object changes as they're included in that PR


Preview | Diff

@lukewarlow lukewarlow requested a review from koto May 14, 2024 15:04
@koto
Copy link
Member

koto commented May 14, 2024

Do you want to hold on merging of this? For now the PR is in a draft status -- perhaps it makes sense to reference it, but not remove the algorithm here until the merge happens?

@lukewarlow
Copy link
Member Author

So for the DOM section we just point to the PRs and don't have the changes in this spec too. I was going to do the same for this. If you think it's best to wait (the reason it's draft is an underlying PR needs merging first) then that's fine. :)

@koto koto merged commit be22ef9 into w3c:main May 14, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request May 14, 2024
…pstream PR (#514)

SHA: be22ef9
Reason: push, by koto

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@lukewarlow lukewarlow deleted the csp-upstream branch May 14, 2024 15:28
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