-
Notifications
You must be signed in to change notification settings - Fork 48
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
Split up jlmkr.py into smaller files as-is #233
Conversation
Did you try this? This used to work on SCALE but perhaps it no longer works... |
Heck yes. See my Jira. I'd actually gotten a handful of commits into dissection using only an editor and a Makefile on macOS. But then doubled back to test launch, needed a ZFS dataset your other preconditions, and realized that I should get the final target build system in place first. For the venv I exhausted a bunch of potential workarounds using pipx and such, then submitted NAS-130029. For orchestration I surveyed current practices, got up to speed on PEP 517/518, and circled in on Hatch to keep ceremony to a minimum. Since then I've been working on configuring that and the associated GitHub build action. After that I'll be mercifully back to parceling out the script again. Apologies for all the yak shaving. |
That's the big one with all the moving parts. Now on to the Python script itself. |
Per PEP 8
839d45b
to
c72bfa2
Compare
91e5f6b
to
d98ccf8
Compare
d98ccf8
to
1679d27
Compare
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've fixed some more imports and added a new entrypoint script so the tests can run again (running the test caught this import error). Although they are rudimentary let's strive to keep running the tests we have. From the last run I see the "jailmaker" dir safety check needs changes.
The modularization is a good first step but hasn't yet resulted in working code. Looking forward to seeing the tests green again. 😃
I'd like to keep it simple. For now no Hatch. |
🥳 |
Please have a look at the updated README, specifically this bit about configuring the |
WOOOOOT! 🤩
Yespleez. I, too, found this to be a linchpin to get tests back online. PR forthcoming. FYI, I envision a fallback progression for which exact policy details will be ripe for discussion and adjustment. |
For now I'd be happy with just the env var mechanics. |
Stage One of a three-stage modularization plan. See discussion.