Skip to content

Pass-down -downgrade-typecheck-interface-error to interface verification job #903

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

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

nkcsgexi
Copy link
Contributor

The interface verification phase should downgrade errors to warnings as an
intermediate step to stage it in. This won't fail the build if interface errors
occur but it should help users verify project-side fixes.

@nkcsgexi nkcsgexi requested a review from xymus November 10, 2021 17:55
@nkcsgexi
Copy link
Contributor Author

@swift-ci please test

@xymus
Copy link
Contributor

xymus commented Nov 10, 2021

Can we pass it down only when the verification is not explicitly requested as discussed earlier today?

@nkcsgexi nkcsgexi force-pushed the use-warning-interface-verify branch from 3a16b11 to 9885763 Compare November 10, 2021 23:08
@nkcsgexi
Copy link
Contributor Author

@swift-ci please test

@nkcsgexi
Copy link
Contributor Author

@xymus sure, additional check added.

…ion job

The interface verification phase should downgrade errors to warnings as an
intermediate step to stage it in. This won't fail the build if interface errors
occur but it should help users verify project-side fixes.
@nkcsgexi nkcsgexi force-pushed the use-warning-interface-verify branch from 9885763 to 504d9d7 Compare November 11, 2021 18:02
@nkcsgexi
Copy link
Contributor Author

@swift-ci please test

@xymus
Copy link
Contributor

xymus commented Nov 11, 2021

Thanks for the env var support!

@nkcsgexi nkcsgexi merged commit 1dbe411 into swiftlang:main Nov 11, 2021
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