-
Notifications
You must be signed in to change notification settings - Fork 6.1k
New test setting types: enum list/enum set #16015
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
base: refactor-soltest-error-handling
Are you sure you want to change the base?
Conversation
5bba037
to
6d69aad
Compare
fc66d33
to
0c1a327
Compare
0c1a327
to
caeaa48
Compare
6d69aad
to
94e28fe
Compare
This pull request is stale because it has been open for 14 days with no activity. |
This pull request is stale because it has been open for 14 days with no activity. |
This pull request is stale because it has been open for 14 days with no activity. |
This pull request is stale because it has been open for 14 days with no activity. |
This pull request is stale because it has been open for 14 days with no activity. |
This pull request is stale because it has been open for 14 days with no activity. |
Depends on #16014.
A small quality-of-life improvement extracted from #16012. The PR is yet another case (after #15803) where a list setting would have been useful so I added one.
May also come in handy in #15819, though for that a list of ranges would be better.
The PR also adjusts the handling of the
outputs
setting inObjectCompilerTest
to be consistent with what the evmasm test case does (i.e. no hiding of empty ones).