Skip to content

Conversation

@stzn
Copy link
Owner

@stzn stzn commented Sep 6, 2025

Closes #648

@stzn stzn requested a review from Copilot September 6, 2025 23:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR corrects the documentation about the names: argument requirement for different macro roles in Swift. The change clarifies when accessor macros need to specify the names: argument and explains the restrictions for property observer generation.

Key Changes

  • Clarified that peer and member macro roles require the names: argument
  • Specified that accessor macros only need names: when generating willSet or didSet property observers
  • Added explanation that accessor macros generating property observers cannot add other accessors

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@stzn stzn merged commit 8784f66 into master Sep 13, 2025
1 check passed
@stzn stzn deleted the fix-macro-role branch September 13, 2025 22:55
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.

マクロのroleの修正

1 participant