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

Use var instead of let for Endpoint.Response properties #42

Open
emma-k-alexandra opened this issue Feb 7, 2022 · 0 comments
Open

Use var instead of let for Endpoint.Response properties #42

emma-k-alexandra opened this issue Feb 7, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@emma-k-alexandra
Copy link
Owner

Currently I think the usage of let is a bit onerous in the data coming back from the API. I think it's fine if they're moved to vars, and could make some development a bit easier.

There currently isn't a great mechanism to modify the data you receive from the API without creating a brand new struct, which can be tedious. If your project uses the data structures provided by this package, and it needs to modify the data it gets in some way, I think it should be able to do that.

@emma-k-alexandra emma-k-alexandra added enhancement New feature or request help wanted Extra attention is needed labels Feb 7, 2022
@emma-k-alexandra emma-k-alexandra self-assigned this Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant