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

added Warning options to the list of scala3 compiler options#2959

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

Open
rssh wants to merge3 commits intoscala:main
base:main
Choose a base branch
Loading
fromrssh:patch-1

Conversation

@rssh
Copy link

No description provided.

@rssh
Copy link
Author

Occasionally, this will address#2940


| 3.3.x| description|
|-|-|
|`-Wunused:imports`| Warn about unused imports|
Copy link
Contributor

Choose a reason for hiding this comment

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

weren't all -Wunused present in Scala 2? The same for value-discard.Wnonunit-statement seems to be new, same forWimplausible-patterns and some others.

ScalaSettings.scala#L169-L170
https://docs.scala-lang.org/overviews/compiler-options/

Copy link
Member

Choose a reason for hiding this comment

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

Scala 2 has-Wnonunit-statement

szymon-rd reacted with thumbs up emoji
Copy link
Author

Choose a reason for hiding this comment

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

Ok, deleted options that were presented in scala-2

By the way, we need another page with all scala3 options in some other document, but this would be another issue.

Copy link
Author

Choose a reason for hiding this comment

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

But -Wnounit-statement for scala2 is not documented inhttps://docs.scala-lang.org/overviews/compiler-options/

Copy link
Member

@SethTisueSethTisueMar 11, 2024
edited
Loading

Choose a reason for hiding this comment

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

That page has long been in a grievous state of unmaintainedness, I'm ashamed to say:#1711

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@SethTisueSethTisueSethTisue left review comments

+1 more reviewer

@szymon-rdszymon-rdszymon-rd left review comments

Reviewers whose approvals may not affect merge requirements

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

@rssh@SethTisue@szymon-rd

[8]ページ先頭

©2009-2025 Movatter.jp