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(cli): ♻️ Remove tempfile usage for better CI compatib… #24

Closed
wants to merge 2 commits into from

Conversation

SpiralOutDotEu
Copy link
Owner

Summary

Refactored the test files to remove the usage of the tempfile crate, addressing the failure in GitHub Actions. The tests now utilize the current working directory for file operations, ensuring better compatibility with CI environments.

Changes

  • Removed tempfile usage in compile_command_tests.rs and circuit_command_tests.rs.
  • Removed tempfile dependency from Cargo.toml and Cargo.lock
  • Utilized the current working directory for file operations in tests.
  • Added cleanup functions to remove any files and directories created during tests.

Check

  • Tests pass in the CI environment.
  • No leftover files after tests are executed.

@SpiralOutDotEu
Copy link
Owner Author

Test fails again

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.

None yet

1 participant