Skip to content

Conversation

@rxwei
Copy link
Contributor

@rxwei rxwei commented May 11, 2023

Cherry-pick #65814


  • Explanation: Improve an incorrect diagnostic message that misleads people to using -load-plugin-library to load plugin libraries, whereas most developers should actually be using a Swift package dependency for a self-defined or open-source macro implementation.
  • Scope: One type checker diagnostic message
  • Risk: Low
  • Testing: Updated tests for the new diagnostic
…ang#65814) We should no longer say "macro implementation must be provided via '-load-plugin-library'" as there are multiple ways to pick up a macro dependency. Here's an improved version: > warning: external macro implementation type 'MissingModule.MissingType' could not be found for macro 'missingMacro1'; the type must be public and provided by a macro target in a Swift package, or via '-plugin-path' or '-load-plugin-library'
@rxwei rxwei requested a review from a team as a code owner May 11, 2023 04:54
@rxwei rxwei changed the title [Macros] Improve diagnostics for missing macro implementation [5.9] [Macros] Improve diagnostics for missing macro implementation May 11, 2023
@rxwei
Copy link
Contributor Author

rxwei commented May 11, 2023

@swift-ci please test

@rxwei rxwei merged commit 385516a into swiftlang:release/5.9 May 11, 2023
@rxwei rxwei deleted the missing-impl-missing-diag-5.9 branch May 11, 2023 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants