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: further cleanup agave #587

Merged
merged 4 commits into from
Mar 23, 2024
Merged

Conversation

akshaynexus
Copy link
Contributor

just cleanup code and log data better

@akshaynexus akshaynexus reopened this Mar 23, 2024
@akshaynexus
Copy link
Contributor Author

looks like actions is not working well,will check and fix that shortly @SunWeb3Sec

@SunWeb3Sec
Copy link
Owner

@akshaynexus
Please fix command in ci. Thanks a lot!
Error:
fatal: ambiguous argument 'origin/main': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:

@akshaynexus
Copy link
Contributor Author

@SunWeb3Sec Check pr #588 and merge it ,then run the action again

@akshaynexus
Copy link
Contributor Author

@SunWeb3Sec The CI Test passes now should be good to go for future PRS to auto test and see if tests pass for new changes of any _exp.sol file

@SunWeb3Sec
Copy link
Owner

Super! Thanks @akshaynexus

@SunWeb3Sec SunWeb3Sec merged commit 380c06b into SunWeb3Sec:main Mar 23, 2024
1 check passed
@akshaynexus akshaynexus deleted the cleanup-agave-2 branch May 11, 2024 13:52
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

2 participants