Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Comments

Close side panel

Added options for mutable properties in classes and structs#17

Merged
kean merged 1 commit intoCreateAPI:mainCreateAPI/CreateAPI:mainfrom
JanC:16-mutable-propertiesJanC/CreateAPI:16-mutable-propertiesCopy head branch name to clipboard
Feb 5, 2022
Merged

Added options for mutable properties in classes and structs#17
kean merged 1 commit intoCreateAPI:mainCreateAPI/CreateAPI:mainfrom
JanC:16-mutable-propertiesJanC/CreateAPI:16-mutable-propertiesCopy head branch name to clipboard

Conversation

@JanC
Copy link
Contributor

@JanC JanC commented Feb 5, 2022 •
edited
Loading

As discussed in #16, I added an option to generate properties either as let or var.

You mentioned that it would be good to have some granularity so I added an option for classes and structs

  # If disabled, will use let properties in struct entities
  isGeneratingMutableStructProperties: false
  # If disabled, will use let properties in class entities
  isGeneratingMutableClassProperties: false

This also allows to keep the current behaviour where classes have let and structs have var .

Let me know if you have other suggestions

@kean kean merged commit a057def into CreateAPI:main Feb 5, 2022
@JanC JanC deleted the 16-mutable-properties branch February 5, 2022 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JanC @kean
Morty Proxy This is a proxified and sanitized view of the page, visit original site.