Skip to content

Swift: Remove workaround from swift/unsafe-js-eval#11715

Merged
MathiasVP merged 1 commit intogithub:mainfrom
geoffw0:jsevalcleanup
Dec 15, 2022
Merged

Swift: Remove workaround from swift/unsafe-js-eval#11715
MathiasVP merged 1 commit intogithub:mainfrom
geoffw0:jsevalcleanup

Conversation

@geoffw0
Copy link
Copy Markdown
Contributor

@geoffw0 geoffw0 commented Dec 15, 2022

Remove one of the workarounds from swift/unsafe-js-eval, that is covered by our general taint modelling since #11185 was merged. The query already has adequate test coverage for these cases.

@d10c

@geoffw0 geoffw0 added no-change-note-required This PR does not need a change note Swift labels Dec 15, 2022
@geoffw0 geoffw0 requested a review from a team as a code owner December 15, 2022 16:45
Copy link
Copy Markdown
Contributor

@d10c d10c left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@MathiasVP MathiasVP merged commit 801f4bc into github:main Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-change-note-required This PR does not need a change note Swift

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants