-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add local file system functionality to all tools #411
Add local file system functionality to all tools #411
Conversation
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 reviewed the projects part and most of the stores, but I'm getting bogged down in the UI. Can you break this into two PRs - one that updates the project layout and structure, and a second that does the file system user interface?
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.
Again, I was able to review changes in projects/ and most of simulator/ and components, but please pull out (at least) web/ and related changes to a separate PR.
Looks like there's maybe four changes in here?
- setStatus to Action
- Project test structure
2b. Project files with updated load instructions - Web shell to create files
Sorry I missed your last comment about this. Will do. |
fd76718
to
7220a12
Compare
1d2dd5c
to
fe7a17c
Compare
components/src/stores/asm.store.ts
Outdated
console.log(path); | ||
console.log(await fs.scandir("/")); |
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.
(nit) Remove
simulator/src/test/cputst.ts
Outdated
this.cpu = new CPU({ ROM: rom }); | ||
this.reset(); | ||
} | ||
|
||
override async step() { | ||
if (!this.hasLoad && !this.fileLoaded) { |
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.
(opt) De Morgan's for readability?
if (!(this.hasLoaded || this.fileLoaded))
c6fbf09
to
d3cc013
Compare
Closes #352