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

chore: correct README link due to changes from PR #3266 #3600

Merged
merged 1 commit into from
Jan 26, 2025

Conversation

CodeDragonVN
Copy link
Contributor

Summary

Due to the refactor of the light client directory structure from pull request #3266 the link ./light-clients needs to update to ./cosmwasm/ibc-union/light-clients

Copy link

vercel bot commented Jan 21, 2025

@CodeDragonVN is attempting to deploy a commit to the unionbuild Team on Vercel.

A member of the Team first needs to authorize it.

@benluelo
Copy link
Contributor

thanks for the PR! please reword your commit to follow our contributing guidelines.

@CodeDragonVN CodeDragonVN force-pushed the main branch 2 times, most recently from 3199446 to 0089e18 Compare January 22, 2025 13:08
@CodeDragonVN
Copy link
Contributor Author

thanks for the PR! please reword your commit to follow our contributing guidelines.

@benluelo Could you please check if the commit aligns with the guidelines. Thank you.

Copy link
Contributor

@benluelo benluelo left a comment

Choose a reason for hiding this comment

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

looks great, thanks!

unfortunately hit again by our commitlint: https://github.com/unionlabs/union/actions/runs/12909015036/job/35997562303?pr=3600

ensure that the footer lines are <= 100 characters long.

@CodeDragonVN
Copy link
Contributor Author

@benluelo my bad, updated the footer to be under 100 characters.

@benluelo benluelo merged commit 3370db7 into unionlabs:main Jan 26, 2025
104 of 105 checks passed
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.

2 participants