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.

Sign up for GitHub

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

Jump to bottom

add test for modules in modules #36

Open

yaahc wants to merge 2 commits into main from module-in-module

Collaborator

yaahc commented Feb 23, 2024

No description provided.


          add test for modules in modules

yaahc force-pushed the module-in-module branch 2 times, most recently from 728db10 to 178b13b Compare

February 23, 2024 19:59

yaahc commented

View reviewed changes

src/codegen.rs Outdated Show resolved Hide resolved

yaahc force-pushed the module-in-module branch from 178b13b to 3bcaa38 Compare

February 23, 2024 20:12

yaahc commented

View reviewed changes

tests/integration_tests.rs Outdated Show resolved Hide resolved

yaahc force-pushed the module-in-module branch 7 times, most recently from c71399c to b2bb9ed Compare

February 28, 2024 20:17


          Add support for calls with more than two path segments

38a7c78

* Associate call resolution with outermost NodeId for namespaced calls
* update codegen to check call resolution on outermost namespaced lookup

yaahc force-pushed the module-in-module branch from b2bb9ed to 38a7c78 Compare

February 28, 2024 20:19

sophiajt reviewed

View reviewed changes

tests/integration/modules/deep.june

+              use deep2;
+              fun main() {
+                  deep2::deep3::say_hello()

Owner

sophiajt Feb 28, 2024

For this, I was wondering if we would want deep2 to be a directory we traverse to get to deep3.

Probably worth brainstorming whether files are the only modules or if directories can be too.

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