Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Js.Dict.t -> dict#822

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
cknitt merged 1 commit intomainfromdict
Dec 10, 2024
Merged

Js.Dict.t -> dict#822

cknitt merged 1 commit intomainfromdict
Dec 10, 2024

Conversation

@cknitt
Copy link
Member

No description provided.

@Freddy03h
Copy link
Member

You want to remove all ref toJs module before the0.75 release?

@cknitt
Copy link
MemberAuthor

cknitt commentedDec 10, 2024
edited
Loading

Where possible, yes.
There may be some follow-up PRs to this one. 🙂

Freddy03h reacted with thumbs up emoji

@Freddy03h
Copy link
Member

Ok, and you also want to addCore ?

@cknitt
Copy link
MemberAuthor

No, this would be hard, because right now it is an extra dependency, but in v12 it will be part of therescript npm package.

Freddy03h reacted with thumbs up emoji

@cknittcknitt merged commit5141687 intomainDec 10, 2024
2 checks passed
@cknittcknitt deleted the dict branchDecember 10, 2024 13:00
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@Freddy03hFreddy03hFreddy03h approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cknitt@Freddy03h

[8]ページ先頭

©2009-2025 Movatter.jp