-
Notifications
You must be signed in to change notification settings - Fork 35
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
Folder/Package Renaming #16
Comments
I like the division |
To avoid circular dependencies here is an outline of the updated package ordering:
|
Updated spec in #25 |
Currently the bulk of the application is under the package types. The name "types" does not accurately represent all the files it contains. I think we should split this package into 2. 1 folder/package containing files for validation and low level database changes (mapper) and the other folder/package for higher level application changes and structure definitions.
For example we could have the packages auth and db:
package auth contains:
package db contains:
To work with the current version of the cosmos sdk we should add a errors.go file to define the codespace our errors will function in as outlined here. Notice that none of the files in package auth need to return an sdk.Error.
Thoughts?
The text was updated successfully, but these errors were encountered: