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

add binary literals support #3543

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Conversation

kasiaMarek
Copy link
Contributor

Copy link
Collaborator

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

Lgtm! @kitbellew do you see any issues? Should be a noop change for scalafmt

@kitbellew
Copy link
Contributor

kitbellew commented Feb 7, 2024

i looked at the two threads on scala repo, it seems that the feature is only in 2.13 but not in dotty (and so definitely not in 3.0-2 but could be added later to whatever next milestone is).

should we disable this flag in scala30 dialect (which likely derives from scala213) and perhaps reenable in scala3future?

@@ -30,7 +30,7 @@ The exact test command to run tests depends on what you are working on:
- `testsJVM/slow:test`: run slow integration tests for semantic APIs.
- `testsJVM/all:test`: run all tests, both fast and slow.

Tips to make your edit/test/debug worflow as productive as possible:
Tips to make your edit/test/debug workflow as productive as possible:
Copy link
Contributor

Choose a reason for hiding this comment

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

good change but unrelated. why not submit as a separate pr?

@kasiaMarek
Copy link
Contributor Author

kasiaMarek commented Feb 7, 2024

should we disable this flag in scala30 dialect (which likely derives from scala213) and perhaps reenable in scala3future?

I think that is what I did.

@tgodzik tgodzik merged commit 9df6bbf into scalameta:main Feb 7, 2024
24 checks passed
@Kordyjan
Copy link

It is worth noting that this is not something that is present in any released version of Scala 3. It will become part of the language in 3.5.

@tgodzik
Copy link
Collaborator

tgodzik commented Feb 16, 2024

It's currently in the Scala3Future dialect, so this should be safe

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.

None yet

4 participants