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

Fix test failures due to missing .img suffix #78

Merged
merged 1 commit into from
Oct 19, 2024

Conversation

kapkekes
Copy link
Contributor

#76 silently broke tests, as it made binary the default mode for image generation at cocas; this PR adds a .img suffix for generated paths, so cocas outputs images in Logisim format.

@kapkekes kapkekes added the fix Fixes something label Oct 19, 2024
@kapkekes kapkekes requested a review from Intelix8996 October 19, 2024 18:04
@kapkekes kapkekes self-assigned this Oct 19, 2024
@Intelix8996 Intelix8996 merged commit 1ce2eec into master Oct 19, 2024
2 checks passed
@Intelix8996 Intelix8996 deleted the tests-missing-suffix branch October 19, 2024 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes something
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants