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

📝 Updated stylex.create docs #338

Closed
wants to merge 3 commits into from
Closed

Conversation

amjadorfali
Copy link

What changed / motivation ?

Documentation changes, added a warning for typescript users using TS < 5

Linked PR/Issues

Fixes #327

Additional Context

Referenced typescript limitation in using the new const keyword

Pre-flight checklist

- Referenced typescript limitation in using the new `const` keyword
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 11, 2024
@amjadorfali
Copy link
Author

@nmn Here's the PR, I wasn't sure about the references but I added them anyway. Please let me know if I missed anything

Copy link
Contributor

@nmn nmn left a comment

Choose a reason for hiding this comment

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

Let's also just wait since I might be able to solve this problem.

apps/docs/docs/api/javascript/create.mdx Outdated Show resolved Hide resolved
@@ -25,6 +25,9 @@ The recommended way to use StyleX in development and production is with the
build-time compiler.

### Development
:::danger TypeScript Limitation
Copy link
Contributor

Choose a reason for hiding this comment

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

The title should be changed. I'll think about what it should be.

@nmn
Copy link
Contributor

nmn commented Mar 2, 2024

I'm closing this PR. I'll update the docs with the minimum typescript requirement.

@nmn nmn closed this Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Safe typings for TypeScript v4
4 participants