Skip to content
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

fix: Selection set initializer reserved parameter name #257

Merged

Conversation

calvincestari
Copy link
Member

@calvincestari calvincestari commented Feb 5, 2024

Fixes apollographql/apollo-ios#3330.

Adds an underscored alias name to initializer parameters that would conflict with any Swift reserved names. This has only been an issue with self specifically within initializers but this enforces the same behaviour for any of the Swift reserved keywords.

@calvincestari calvincestari requested a review from a team as a code owner February 5, 2024 22:17
Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for eclectic-pie-88a2ba canceled.

Name Link
🔨 Latest commit e9a948b
🔍 Latest deploy log https://app.netlify.com/sites/eclectic-pie-88a2ba/deploys/65c15e8b9d789900089ecae3

Copy link
Contributor

@AnthonyMDev AnthonyMDev left a comment

Choose a reason for hiding this comment

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

This is a great solution!

@apollographql apollographql deleted a comment from Danyalkasiri Feb 5, 2024
@apollographql apollographql deleted a comment from Danyalkasiri Feb 5, 2024
@calvincestari calvincestari merged commit a335842 into main Feb 5, 2024
20 checks passed
@calvincestari calvincestari deleted the fix/selection-set-initializer-reserved-parameter-name branch February 5, 2024 23:01
BobaFetters pushed a commit to apollographql/apollo-ios-codegen that referenced this pull request Feb 5, 2024
BobaFetters pushed a commit that referenced this pull request Feb 5, 2024
c844d744 fix: Selection set initializer reserved parameter name (#257)

git-subtree-dir: apollo-ios-codegen
git-subtree-split: c844d7448132d0d993a0729512e088ff3b0a661c
BobaFetters pushed a commit that referenced this pull request Feb 5, 2024
…reserved parameter name

git-subtree-dir: apollo-ios-codegen
git-subtree-mainline: fb53113
git-subtree-split: c844d7448132d0d993a0729512e088ff3b0a661c
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.

SelectionSet Generated Initializers Don't Compile with self Parameter
2 participants