Skip to content

Enable sc:compile for scala.util Package#25651

Open
cheeseng wants to merge 1 commit intoscala:mainfrom
artimahub:feature-enable-stdlib-sc-compile
Open

Enable sc:compile for scala.util Package#25651
cheeseng wants to merge 1 commit intoscala:mainfrom
artimahub:feature-enable-stdlib-sc-compile

Conversation

@cheeseng
Copy link
Copy Markdown
Contributor

Add sc:compile and sc:nocompile to code snippets under scala.util package, add necessary code to make them compile successfully.

@cheeseng cheeseng requested a review from a team as a code owner March 31, 2026 03:55
Comment on lines +65 to +67
* ```scala sc:compile
* import scala.util.Try
*
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wonder if we can use reusable sc-name snippets and hidden snippets like we can do in Markdown scaladoc for these

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@natsukagami that's a good idea, I have made the change and squashed the commits into one.

…used sc-name and sc-hidden for context code, and adjusted the code snippets to get them to compile correctly.
@cheeseng cheeseng force-pushed the feature-enable-stdlib-sc-compile branch from 3d45337 to 680a398 Compare April 3, 2026 13:30
@SethTisue SethTisue self-assigned this Apr 3, 2026
* for (...) {
* if (...) break
* for (i <- 1 to 10) {
* if (i == 5) break
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I suggest using Scala 3 style for...do and if...then

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.

3 participants