这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@frankcalise
Copy link
Contributor

@frankcalise frankcalise commented May 15, 2023

Please verify the following:

  • yarn test jest tests pass with new tests, if relevant

Describe your PR

  • Closes Better types for $sizeStyles #2309
  • Upgrades TS version to match latest VS Code internal version for now
  • Upgrades typescript-eslint and prettier plugins to deal with satisfies properly

Images

Now you get typechecking:
image

And still typeahead (whicih as TextStyle originally accomplished)
image

@frankcalise frankcalise marked this pull request as ready for review May 22, 2023 19:59
Copy link
Contributor

@joshuayoes joshuayoes left a comment

Choose a reason for hiding this comment

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

I've done this a few times on other projects. This also has the nice advantage of upgrading typescript. Looks good!

@frankcalise frankcalise merged commit 2e96d78 into master May 23, 2023
@frankcalise frankcalise deleted the fix/2309 branch May 23, 2023 11:02
infinitered-circleci pushed a commit that referenced this pull request May 23, 2023
## [8.8.2](v8.8.1...v8.8.2) (2023-05-23)

### Bug Fixes

* **boilerplate:** Podfile update for hermes-engine ([#2442](#2442) by [@frankcalise](https://github.com/frankcalise)) ([e118b11](e118b11))
* **boilerplate:** update Text sizeStyles to use satisfies ([#2437](#2437) by [@frankcalise](https://github.com/frankcalise)) ([2e96d78](2e96d78))
@infinitered-circleci
Copy link
Collaborator

🎉 This PR is included in version 8.8.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better types for $sizeStyles

5 participants