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

ScalaCLI should accumulate warnings about using experimental directives #2294

Closed
lwronski opened this issue Jul 15, 2023 · 1 comment · Fixed by #2376
Closed

ScalaCLI should accumulate warnings about using experimental directives #2294

lwronski opened this issue Jul 15, 2023 · 1 comment · Fixed by #2376
Labels
enhancement New feature or request

Comments

@lwronski
Copy link
Contributor

Is your feature request related to a problem? Please describe.
1.0.2

Describe the solution you'd like
Generated warnings should be cumulative, containing every experimental directives in a single warning message

Describe alternatives you've considered

For example, when using publish directives, ScalaCLI generates multiple warnings with almost identical messages:

The `//> using publish.developer "scala-cli"` directive is experimental.
Please bear in mind that non-ideal user experience should be expected.
If you encounter any bugs or have feedback to share, make sure to reach out to the maintenance team at https://github.com/VirtusLab/scala-cli
The `//> using publish.license "scala-cli"` directive is experimental.
Please bear in mind that non-ideal user experience should be expected.
If you encounter any bugs or have feedback to share, make sure to reach out to the maintenance team at https://github.com/VirtusLab/scala-cli
The `//> using publish.computeVersion "git:tag"` directive is experimental.
Please bear in mind that non-ideal user experience should be expected.
If you encounter any bugs or have feedback to share, make sure to reach out to the maintenance team at https://github.com/VirtusLab/scala-cli
The `//> using publish.repository "central"` directive is experimental.
Please bear in mind that non-ideal user experience should be expected.
If you encounter any bugs or have feedback to share, make sure to reach out to the maintenance team at https://github.com/VirtusLab/scala-cli

Instead of the previous warnings, the output should look like this:

The directives:
  `//> using publish.developer "scala-cli"` 
  `//> using publish.license "scala-cli"`
  `//> using publish.computeVersion "git:tag"` 
  `//> using publish.repository "central"` are experimental.
Please bear in mind that non-ideal user experience should be expected.
If you encounter any bugs or have feedback to share, make sure to reach out to the maintenance team at https://github.com/VirtusLab/scala-cli
@MaciejG604
Copy link
Contributor

Done in #2376

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants