-
Notifications
You must be signed in to change notification settings - Fork 316
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
Fix a Crash when Project Names are Shadowed #1571
Conversation
1bf6e7a
to
e312c29
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have only looked at the legal review so far, but there are already a few things that I think are worth resolving, so I'm submitting these comments already.
I'll look at other parts and submit them as a separate comment.
tools/legal-review/Standard/com.ibm.icu.icu4j-67.1/copyright-keep
Outdated
Show resolved
Hide resolved
tools/legal-review/Standard/org.xerial.sqlite-jdbc-3.34.0/copyright-keep
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more comment inline.
Also I have noticed by doing a search in the repo state as of the latest commit in this PR, that the test for the Rust lexer enso_lexer_combined.rs
still uses the old from Base import all
. Is this deliberate?
I think the release.yml
workflow needs to have the verifyGeneratedPackage
entries updated because they now refer to inexistent directories. Given that the changes are big, before merging this PR a test release should be done on staging to ensure that the workflow is still working correctly.
Other than that it looks good.
engine/runtime/src/main/java/org/enso/interpreter/runtime/Context.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me (but still the staging run shall be checked before merging)
engine/polyglot-api/src/test/scala/org/enso/polyglot/ModuleManagementTest.scala
Show resolved
Hide resolved
9b8dec9
to
41107ad
Compare
264d274
to
81d2808
Compare
Pull Request Description
As in the title. In addition, it namespaces all of the standard library packages under
Standard
.Closes #1558.
Important Notes
Found a bug in imports resolution, but that will be fixed in a separate task.
Checklist
Please include the following checklist in your PR: