Skip to content

Swift: add an internal query-suite for listing all the compiler errors#11181

Merged
MathiasVP merged 3 commits intomainfrom
alexdenisov/extractor-errors-suite
Nov 9, 2022
Merged

Swift: add an internal query-suite for listing all the compiler errors#11181
MathiasVP merged 3 commits intomainfrom
alexdenisov/extractor-errors-suite

Conversation

@AlexDenisov
Copy link
Copy Markdown
Contributor

No description provided.

@AlexDenisov AlexDenisov requested a review from a team as a code owner November 9, 2022 11:06
@github-actions github-actions Bot added the Swift label Nov 9, 2022
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Comment thread swift/ql/lib/codeql/swift/elements/Diagnostics.qll Fixed
Copy link
Copy Markdown
Contributor

@MathiasVP MathiasVP left a comment

Choose a reason for hiding this comment

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

One small comment. Otherwise, LGTM!

If you do end up changing the id it would also be awesome to add in those missing this qualifiers that Code Scanning is complaining about. People really want those implicit thiss to disappear from our repo 😂

Comment thread swift/ql/src/diagnostics/internal/CompilerErrors.ql Outdated
@AlexDenisov AlexDenisov requested a review from MathiasVP November 9, 2022 11:52
Copy link
Copy Markdown
Contributor

@MathiasVP MathiasVP 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 50c2683 into main Nov 9, 2022
@MathiasVP MathiasVP deleted the alexdenisov/extractor-errors-suite branch November 9, 2022 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants