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.

Sign up for GitHub

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

Jump to bottom

--fix Command Boolean Handling #3052

Open
s0kil opened this issue Jul 25, 2024 · 0 comments · May be fixed by #3414
Open

--fix Command Boolean Handling #3052

s0kil opened this issue Jul 25, 2024 · 0 comments · May be fixed by #3414
Labels
bug Something isn't working experimental Tickets tied to experimental features. subcommand:fix issues tied to the `fix` sub-command.
Milestone
v1.6.0

Comments

@s0kil
Copy link

s0kil commented Jul 25, 2024

Version(s)
Scala CLI version: 1.4.1
Scala version (default): 3.4.2

Describe the bug
When executing scala-cli --power fix ., boolean values in directives are written as a string, example, true is "true"

To Reproduce
Add, //> using jsEmitSourceMaps true to your project.scala and execute the command above.

Expected behavior
Boolean values should not be wrapped in a string
@s0kil s0kil added the bug Something isn't working label Jul 25, 2024
@Gedochao Gedochao added subcommand:fix issues tied to the `fix` sub-command. experimental Tickets tied to experimental features. labels Jul 26, 2024
@Gedochao Gedochao added this to the v1.6.0 milestone Jan 8, 2025
@Gedochao Gedochao linked a pull request Jan 10, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working experimental Tickets tied to experimental features. subcommand:fix issues tied to the `fix` sub-command.
Projects
None yet
Milestone
v1.6.0
Development

Successfully merging a pull request may close this issue.

fix built-in rules: don't wrap directive values in double quotes if not necessary Gedochao/scala-cli
2 participants
@Gedochao @s0kil