-
Notifications
You must be signed in to change notification settings - Fork 17
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
Restructure #50
base: master
Are you sure you want to change the base?
Restructure #50
Conversation
CI is failing build. |
The tests have not been modified yet, I mainly need people to test the restructure and suggest any changes they want to see since I rarely use armake2 personally |
It's very hard to comment on this PR as a whole since it contains so many changes in a single commit, so it would probably be easier to split this up into multiple PRs. On some of the actual changes:
|
This changes so much that might not be possible. Separate issues could be used to discuss different parts of the restructure |
There really needs to be a confirm button for closing a PR
I'm fine with those not being public, they were basically all that was public in master so I left them public |
I believe they were originally marked as I'll try to play around with some of the changes myself this week; I might try to seperate some changes for overview. |
This turns armake2 into a proper Rust library.
It is still missing a few small things from
master
but is ready for testing and any suggested tweaksNotes
signing
is now a feature, meaning armake2 can be used without openssl.