Skip to content
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

Rust Rewrite #215

Draft
wants to merge 92 commits into
base: main
Choose a base branch
from
Draft

Rust Rewrite #215

wants to merge 92 commits into from

Conversation

mindstorm38
Copy link
Owner

@mindstorm38 mindstorm38 commented May 5, 2024

This is a tracking pull request for the work-in-progress Rust rewrite, this will take a long, long time!

If you want to give feedback on the concept, not the code, please go check #196.

Feel free to review the code!

TODO:

  • Important commit to also implement (only for modern versions): ea5cee6
  • JVM policy argument and JVM arguments as a whole

@mindstorm38 mindstorm38 linked an issue May 5, 2024 that may be closed by this pull request
Repository owner deleted a comment from codecov bot May 5, 2024
Repository owner deleted a comment from codecov bot May 5, 2024
Copy link

codecov bot commented May 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@eb690c9). Learn more about missing BASE report.

Current head cf57c77 differs from pull request most recent head 8c64ba3

Please upload reports for the commit 8c64ba3 to get more accurate results.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #215   +/-   ##
=======================================
  Coverage        ?   63.60%           
=======================================
  Files           ?       20           
  Lines           ?     3314           
  Branches        ?        0           
=======================================
  Hits            ?     2108           
  Misses          ?     1206           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mindstorm38
Copy link
Owner Author

mindstorm38 commented Dec 14, 2024

I've created a C API crate, mainly as a placeholder for now. If someone wants to work on it, and mainly on the C header for now, feel free, I'll not work on this before finishing the CLI, so feel free to completly change it and propose your own view of it, I'm curious because from my short try I didn't come with a good API imo.

Poke @Ristovski 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tracking Rust rewrite (feedbacks are welcome)
1 participant