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

refactor: library #116

Merged
merged 7 commits into from
Feb 7, 2024
Merged

refactor: library #116

merged 7 commits into from
Feb 7, 2024

Conversation

DaniPopes
Copy link
Member

@DaniPopes DaniPopes commented Feb 6, 2024

  • removed SVM_DATA_DIR in favor of data_dir() function
  • renamed current_version to get_global_version
  • renamed use_version to set_global_version
  • renamed SolcVmError to SvmError
  • renamed lib name svm_lib to svm
  • added version_binary to calculate the solc executable path for a version
  • made patch_for_nixos private
  • made setup_data_dir private
  • made hex_string::serialize add hex prefix to match deserialized version

Rest are non functional changes for performance or readability.

@DaniPopes DaniPopes merged commit efd7d70 into alloy-rs:master Feb 7, 2024
8 checks passed
@DaniPopes DaniPopes deleted the refactorings branch February 7, 2024 01:26
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.

2 participants