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: contract type collision issues from deleted files and empty file issues #1818

Merged

Conversation

antazoey
Copy link
Member

@antazoey antazoey commented Jan 5, 2024

What I did

  • fix: issue where contract type collision occurred from deleted files because of a bad cache
  • fix: issue where empty contracts endlessly compiled

fixes: #1780

How I did it

only add trailing \n if not empty

How to verify it

compile an empty contract more than once and see what happens

Checklist

  • All changes are completed
  • New test cases have been added
  • Documentation has been updated

@antazoey antazoey force-pushed the fix/empty-contracts-infinite-compile branch from 36d4b86 to ca6d8b8 Compare January 5, 2024 14:56
@antazoey antazoey changed the title fix: issue where empty contracts would endlessly compile fix: contract type collision issues from deleted files Jan 5, 2024
@antazoey antazoey changed the title fix: contract type collision issues from deleted files fix: contract type collision issues from deleted files and empty file issues Jan 5, 2024
fubuloubu
fubuloubu previously approved these changes Jan 5, 2024
@antazoey antazoey force-pushed the fix/empty-contracts-infinite-compile branch 3 times, most recently from 4876117 to e61ed6d Compare January 5, 2024 18:02
@antazoey antazoey force-pushed the fix/empty-contracts-infinite-compile branch from 7d87aee to 7f61f5d Compare January 5, 2024 18:35
@antazoey antazoey enabled auto-merge (squash) January 5, 2024 18:58
@antazoey antazoey disabled auto-merge January 5, 2024 18:58
@antazoey antazoey merged commit 3c86b54 into ApeWorX:main Jan 5, 2024
15 checks passed
@antazoey
Copy link
Member Author

antazoey commented Jan 5, 2024

was previously approved and all i did was fight caplog into submitting to me

@antazoey antazoey deleted the fix/empty-contracts-infinite-compile branch January 5, 2024 18:59
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.

Adding interface creates "ContractType collisions"
2 participants