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

Swap package.json's bsconfig.json entry for rescript.json #115

Merged
merged 1 commit into from
May 26, 2024

Conversation

cwstra
Copy link
Contributor

@cwstra cwstra commented May 25, 2024

What it says on the tin; noticed it when I couldn't build after using a fork.

@cometkim
Copy link
Member

Good catch.

Maybe it would be better to publish both rescript.json and bsconfig.json with the same content for compatibility

@cwstra
Copy link
Contributor Author

cwstra commented May 25, 2024

Per @tsnobip in the PR which initially renamed the file, looks like the current version of the library is incompatible with versions <11 anyways?

@cometkim
Copy link
Member

Yes. So, we need to copy content and maintain it for a while

Copy link
Member

@cknitt cknitt left a comment

Choose a reason for hiding this comment

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

Thanks, good catch!

@cknitt cknitt merged commit a6e311a into rescript-lang:master May 26, 2024
1 check passed
cometkim added a commit to cometkim/rescript that referenced this pull request May 26, 2024
but now we have another circular dependency 😅 need to wait the `@rescript/react` release

See rescript-lang/rescript-react#115
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.

3 participants