-
Notifications
You must be signed in to change notification settings - Fork 16
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
Workspace refactor #53
Conversation
Here I separate conjure_oxide into its own package within the workspace and clean up the package structure slightly. |
Not sure where the parsers directory commit comes from... Probably accidentally added to my fork's main at some point. I will also try to set conjure as the default package to run. |
Might need some CI path changes (feel free to move this to a separate issue)
|
(We could always do this after #32 is done) |
ok, this looks fine but I am slightly confused about the relationship between this and #32. is this ready to go? will 32 continue after this is merged? |
It's unrelated. There's some ci refactoring to do here before merge |
(just minor path fixes, shouldn't take long.) |
Fixed the CI paths as said above in #54. (it is based on this branch, so this one needs merging first) |
I can merge this and #54 is everyone is happy? |
No description provided.