Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Types: annotate resource properties #1026

Merged
merged 9 commits into from
Sep 1, 2023

Conversation

richardm-stripe
Copy link
Contributor

@richardm-stripe richardm-stripe commented Aug 24, 2023

Reviewer

This should be ready to go right after #1032 (this is based off that rn)

Summary

  • This won't actually do much good without method return types being declared. (solved by Types for CRUDL methods on parents #1032)
  • Adds class definitions for types that did occur on responses, but did not have any methods defined on them.
  • Adds basic type annotations for simple fields on API responses.
    • No nested objects. These are Any for now.
    • Same for union types.

@richardm-stripe richardm-stripe force-pushed the richardm-resource-property-annotations branch from 7a82c06 to d0b721e Compare August 31, 2023 00:39
@richardm-stripe richardm-stripe changed the base branch from master to richardm-generic-apiresource-parents August 31, 2023 00:43
Base automatically changed from richardm-generic-apiresource-parents to master August 31, 2023 20:10
@richardm-stripe richardm-stripe merged commit 6044d9f into master Sep 1, 2023
@remi-stripe remi-stripe deleted the richardm-resource-property-annotations branch September 28, 2023 23:11
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants