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

Update staging.Compiler.make documentation #19428

Merged
merged 3 commits into from
Jan 18, 2024

Conversation

nicolasstucki
Copy link
Contributor

@nicolasstucki nicolasstucki commented Jan 12, 2024

Fixes #19211
This addresses part of #19170, and #19176

@nicolasstucki
Copy link
Contributor Author

@christophkoch This update of documentation should unblock your use cases.

case ex: java.lang.NoClassDefFoundError =>
throw new Exception(
s"""`scala.quoted.staging.run` failed to load a class.
|The classloader used for the `staging.Compiler` instance might be the correct one.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
|The classloader used for the `staging.Compiler` instance might be the correct one.
|The classloader used for the `staging.Compiler` instance might not be the correct one.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you have lost the not here, everything else looks good to me

@nicolasstucki nicolasstucki merged commit 34628b3 into scala:main Jan 18, 2024
19 checks passed
@nicolasstucki nicolasstucki deleted the fix-19211 branch January 18, 2024 10:35
@Kordyjan Kordyjan added this to the 3.4.1 milestone Feb 14, 2024
WojciechMazur added a commit that referenced this pull request Jun 28, 2024
Backports #19428 to the LTS branch.

PR submitted by the release tooling.
[skip ci]
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.

Staging code example raises compiler warning
3 participants