Skip to content

Added ExtendBaseWith type for Rest API example #69

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

Merged
merged 5 commits into from
Aug 5, 2021

Conversation

JoshuaKGoldberg
Copy link
Collaborator

ExtendsBaseWith applies defaults and/or plugins from its type parameter onto a created type.

I'm not positive I got exactly where to put those types right, so this might be a little off. More test cases should be added if so!

@JoshuaKGoldberg JoshuaKGoldberg force-pushed the rest-api-extend-base-with branch from e3c4e34 to 10b5525 Compare August 2, 2021 07:21
@JoshuaKGoldberg
Copy link
Collaborator Author

@gr2m my local line ending settings on Windows are conflicting with Prettier's settings so I'm sending this in as a separate PR. Otherwise it should be ready for review!

@JoshuaKGoldberg JoshuaKGoldberg requested a review from gr2m August 2, 2021 07:26
@gr2m gr2m self-assigned this Aug 2, 2021
@gr2m
Copy link
Owner

gr2m commented Aug 2, 2021

my local line ending settings on Windows are conflicting with Prettier's settings

just for future reference: you can run npm run lint:fix to make prettier happy. Sorry for the friction, I hope to move the whole code formatting enforcement out of PRs

@JoshuaKGoldberg JoshuaKGoldberg requested a review from gr2m August 5, 2021 00:48
Copy link
Owner

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

🚀

@gr2m gr2m merged commit 1ae82ca into rest-api-client-example Aug 5, 2021
@gr2m gr2m deleted the rest-api-extend-base-with branch August 5, 2021 20:08
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.

2 participants