-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Make command line tests to be able to run on unreleased version #11166
Conversation
The grep command is not working on mac os. |
3b90b6c
to
30529fa
Compare
I rebased on top of #11162 and changed target branch here, so we get confirmation from CI that everything works as expected now. |
30529fa
to
2008bbd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd add a comment explaining the purpose of the sed
command but other than that it seems fine.
Co-authored-by: Kamil Śliwak <kamil.sliwak@codepoets.it>
c3c5f65
to
8da630f
Compare
@cameel I clicked your suggestion and after that I did squash which dismissed your review |
Examples from documentation are run on current build, and unreleased version is strictly less than currentVersion.
This searches for
pragma solidity ^$developmentVersion
orpragma solidity >=$developmentVersion
and replaces it withpragma solidity >0.0.1
Depends on #11162 to be merged because OSX_cli will fail without it. Locally I worked with updates from that PR