Skip to content

Swap out Brython for RustPython#1

Merged
coolreader18 merged 8 commits intochicode:masterfrom
coolreader18:nico-rustpython
Mar 13, 2019
Merged

Swap out Brython for RustPython#1
coolreader18 merged 8 commits intochicode:masterfrom
coolreader18:nico-rustpython

Conversation

@coolreader18
Copy link
Copy Markdown
Contributor

@coolreader18 coolreader18 commented Mar 7, 2019

I'm working on a PR RustPython/RustPython#616 that would add properties to the syntax error returned so that there would be an underline in the editor, but that's not in yet.

@outkine
Copy link
Copy Markdown
Member

outkine commented Mar 7, 2019

Everything seems to be working except for the spritesheet/sprite function. The way that Vuex handles that is honestly so messy that I'm afraid to look into it myself without wasting a couple hours - could you have touched that code in any way?

Comment thread src/nico/src/nico/store.js Outdated

const { language } = state

if (!state.mars) {
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.

The problem is here; I have it only initialize mars once so that listeners don't get added multiple times for each initMars call, so the sprites getter that changes for each run call isn't updated in the context of initMars.

@coolreader18
Copy link
Copy Markdown
Contributor Author

coolreader18 commented Mar 9, 2019

Should be fixed now.

@coolreader18 coolreader18 merged commit 10d85ef into chicode:master Mar 13, 2019
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.

2 participants