Conversation
…ript loading operations are done
… concept of app config so users can set if they want to autoclose the loader of handle it themselves
fpliger
added
tag: environment
| - paths: | ||
| - ./utils.py | ||
| </py-env> | ||
| <py-config> |
marimeireles
left a comment
marimeireles
left a comment
There was a problem hiding this comment.
Okay, sorry for taking so long, this ping got lost in my email box.
I've tested with all examples! Seems to work fine.
One thing though is that I'm getting:
(!) Plugin typescript: @rollup/plugin-typescript TS2488: Type 'unknown' must have a '[Symbol.iterator]()' method that returns an iterator.
src/components/pyconfig.ts: (157:48)
157 this.values = Object.assign({}, ...loadedValues);
~~~~~~~~~~~~
(!) Plugin typescript: @rollup/plugin-typescript: Rollup 'sourcemap' option must be set to generate source maps.
While running the server. I don't know if it's relevant...? Cause as I said all is working smoothly.
| } | ||
|
|
||
| loadRuntimes(){ | ||
| console.log("Initializing runetimes...") |
| @@ -1,76 +1,5 @@ | |||
| <script lang="ts"> | |||
| import Tailwind from './Tailwind.svelte'; | |||
| import { loadInterpreter } from './interpreter'; | |||
There was a problem hiding this comment.
Why was this moved? So it's easier to use with the new array of runtimes that should be on the pyconfig? Or just in general this is the wrong place for this kind of config?
There was a problem hiding this comment.
I removed it because the whole logic was moved to pyconfig.ts and not used here anymore, if needed, other parts can import directly from ./interpreter
The reason I moved to pyconfig was that it helps isolate the whole logic around declaring and initializing runtimes somewhere more contained than just the App. Not sure if pyconfig is the best place for it to be, but it's better than before.
I think there's less and less need for us to have the svelte app... and this also helps us understand how much we depend on it. Atm, afaict, we are only relying on its stores.
|
Thanks @marimeireles , will fix the issues you pointed out before merging. The one related to the |
fpliger
added
status: accepted
3 participants
Fixes #172