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

Add publish workflow, update version and dependencies #7

Merged
merged 7 commits into from
Jan 6, 2024

Conversation

shuhuiluo
Copy link
Contributor

@shuhuiluo shuhuiluo commented Jan 2, 2024

Closes #5.
Add CARGO_REGISTRY_TOKEN to repo secrets before release.

The dotenv dependency and corresponding code blocks for creating client providers using environment variables have been removed. The provider creation logic is simplified by directly calling the `provider()` function of `MAINNET`. This leads to cleaner code and eliminates the need for environment variables for this purpose.
Updated the version of 'aperture-lens' and several dependencies in both 'Cargo.toml' and 'package.json' files for improved performance and security. Also introduced lint and formatting checks in GitHub CI workflows, contributing to code hygiene and standardization.
Bumped up the version of 'aperture-lens' to 0.4.0. Updated the 'viem' dependency to version 2.0.0, along with minor changes in their respective test scenarios.
@shuhuiluo shuhuiluo changed the title Regular maintenance Update version and dependencies Jan 4, 2024
@shuhuiluo shuhuiluo changed the title Update version and dependencies Add publish workflow, update version and dependencies Jan 4, 2024
Copy link
Contributor

@gnarlycow gnarlycow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gnarlycow
Copy link
Contributor

Do you know why the CI hardhat test is trying to connect to Infura even if you seem to have changed it to viem's default mainnet endpoint which is cloudflare-eth.com?

@shuhuiluo
Copy link
Contributor Author

Do you know why the CI hardhat test is trying to connect to Infura even if you seem to have changed it to viem's default mainnet endpoint which is cloudflare-eth.com?

I didn't change the RPC for hardhat tests. Please merge it since I don't have write access.

@shuhuiluo
Copy link
Contributor Author

@gnarlycow Please merge it and publish to crates.io.

@gnarlycow gnarlycow merged commit 3848a13 into Aperture-Finance:main Jan 6, 2024
2 of 4 checks passed
@shuhuiluo shuhuiluo deleted the Lonely-Without-You branch January 6, 2024 00:21
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.

Publish cargo and remove auto-generated bindings
2 participants