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

sdk 47 working branch #1202

Closed
wants to merge 19 commits into from
Closed

Conversation

faddat
Copy link
Contributor

@faddat faddat commented Feb 14, 2023

  • go mod download missing go.sum entries
  • ibc-go v7.0.0

This is another placeholder draft PR for Notional's work to support sdk v0.47.x.

Please consider it to be dangerously incomplete at this time. Don't use it in prod, please and thank you <3

@faddat faddat marked this pull request as ready for review February 14, 2023 20:19
@faddat faddat requested a review from alpe as a code owner February 14, 2023 20:19
@sonarcloud
Copy link

sonarcloud bot commented Feb 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@alpe
Copy link
Contributor

alpe commented Feb 16, 2023

Link: #1198

@faddat
Copy link
Contributor Author

faddat commented Feb 16, 2023

Notional hasn't done like:

@gjermundgaraba
Copy link
Contributor

Maybe I am missing something, but would it not be a good idea to take adding a new CI system in a separate PR? This PR is already huge. If there is an agreement to go from circleci to gh actions (which makes sense to me, personally) this could perhaps get through fairly quickly to main without having to wait for this particular branch.

@alpe alpe added the spike Demo to showcase an idea. label Mar 3, 2023
@faddat
Copy link
Contributor Author

faddat commented Mar 10, 2023

@gjermundgaraba the ci changes were made to improve the repo's portability and forkability

@faddat
Copy link
Contributor Author

faddat commented Mar 15, 2023

I can close this when we've got #1261 merged. Thanks @alpe :)

@faddat
Copy link
Contributor Author

faddat commented Mar 15, 2023

Closed in favor of #1149

@faddat faddat closed this Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spike Demo to showcase an idea.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants