Add support for loading all the files under a directory#1288
Add support for loading all the files under a directory#1288pratik97 wants to merge 1 commit intoBetterThanTomorrow:devfrom
Conversation
|
@pratik97 Is this meant to be a draft / work-in-progress at this point, or is it ready for review? Judging by the PR description lacking information and some checklist items not being completed, I'm guessing it's a WIP. |
|
@bpringe yes, it's a draft, it is only meant for getting early feedback on the path I was going with to implement the load all files feature. I was unsure of the performance considerations and the way we show that the files have been loaded. I had mentioned this on a slack to @PEZ but I should have marked this PR as a WIP as it's far away from review stage 😅 |
|
Is this a proposed solution to #1278 or merely related? I'm unclear whether "loading" means opening an editor or loading the namespace into the repl. |
|
@frankleonrose Judging by the changes it looks like it's about loading the file in the repl. |
|
Yes, it's about loading all the files in REPL using one command. |
What has Changed?
Fixes #
My Calva PR Checklist
I have:
devbranch. (Or have specific reasons to target some other branch.)published. (Sorry for the nagging.)[Unreleased]entry inCHANGELOG.md, linking the issue(s) that the PR is addressing.ci/circleci: buildtest. NB: You need to sign in/up at Circle CI to find the Artifacts tab.The Calva Team PR Checklist:
Before merging we (at least one of us) have:
devbranch (unless reasons).Ping @PEZ, @bpringe