Skip to content

loading js-slang and conductor from sourceacademy/ at npm #3380

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
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

loading js-slang and conductor from sourceacademy/ at npm #3380

wants to merge 2 commits into from

Conversation

martin-henz
Copy link
Member

No description provided.

@martin-henz martin-henz marked this pull request as draft September 24, 2025 08:14
@martin-henz
Copy link
Member Author

test cases fail. investigating...

@s-kybound
Copy link
Member

s-kybound commented Sep 24, 2025 •
edited
Loading

the change of js-slang -> @sourceacademy/js-slang breaks a lot of the frontend. i've renamed all of the references here, but there are probably also other modules, (for example, sourceror) that are affected by this change. only when those are also fixed can the frontend be fixed - will do!

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@martin-henz @s-kybound