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

[Documentation]: Add a README to errors, helping clarify it vs types/errors #17448

Closed
ValarDragon opened this issue Aug 18, 2023 · 0 comments · Fixed by #17867
Closed

[Documentation]: Add a README to errors, helping clarify it vs types/errors #17448

ValarDragon opened this issue Aug 18, 2023 · 0 comments · Fixed by #17867
Assignees
Labels
T:Docs Changes and features related to documentation.

Comments

@ValarDragon
Copy link
Contributor

ValarDragon commented Aug 18, 2023

Summary

Can there be a README.md added to errors?

It was unclear to me until I looked through the source code of what the role of cosmossdk.io/errors was, vs github.com/cosmos-sdk/cosmos/types/errors , and why I had to import both

The Readme could even just point to https://pkg.go.dev/cosmossdk.io/errors , which does a great explanation.

( Though it is still unclear to me if types/errors is planned as a long term for defining the common error types, and why we don't move these common singletons into cosmossdk.io )

@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label Aug 18, 2023
@tac0turtle tac0turtle added T:Docs Changes and features related to documentation. T:Sprint and removed needs-triage Issue that needs to be triaged labels Aug 18, 2023
@github-project-automation github-project-automation bot moved this to 📝 Todo in Cosmos-SDK Aug 18, 2023
@tac0turtle tac0turtle self-assigned this Sep 25, 2023
@github-project-automation github-project-automation bot moved this from 📝 Todo to 👏 Done in Cosmos-SDK Sep 25, 2023
@tac0turtle tac0turtle removed this from Cosmos-SDK Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T:Docs Changes and features related to documentation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants