Deepnest
Deepnest copied to clipboard
Added logic to import multiple files when prompted (fixes issues #31)
Fixes #31
Hopefully, this should improve readability a bit with processing the file. I did two other things:
- Since the code doesn't use the arrow function syntax, I decided to keep it consistent with the rest of the code.
- I relocated the importbutton.className to before and after the end loop. I removed them from the main logic since the current code only took into account one file. It would be best for the button to unlock after all files have been processed.
Looks good to me :) (I have not tested this though)
@deece,
Thanks for taking a look at it. Any idea who has permission to merge? I noticed that there's only one merge from August.
AFAIK only @Jack000 can merge them
should we start a development branch somewhere until jack returns?
I'm in favour of that, without it, projects become abandonware. I'm just about to to the same on GerberTools.
Just make sure you add a new issue to point people to where the new code is.
Is jack also @bmtm ?
Not directly related to the PS. Is there anyword on creating the development branch? It seems like there is other bugs out there that would benefit from this if we are unable to get things merged into master.
Hi @zaped212,
The idea was in favor. However, I'm not aware of any branch being created at the moment. @deece, would one have to fork from this project or is there a way to add a development branch onto this repository?
I'll review & merge PRs on my fork until @Jack000 is available: https://github.com/InfernoEmbedded/Deepnest