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

Internal auditing of our codebase #1831

Closed
10 tasks done
ValarDragon opened this issue Jun 20, 2022 · 1 comment
Closed
10 tasks done

Internal auditing of our codebase #1831

ValarDragon opened this issue Jun 20, 2022 · 1 comment

Comments

@ValarDragon
Copy link
Member

ValarDragon commented Jun 20, 2022

Background

We should go through an internal audit of every module in our codebase over this sprint, aiming to hit our new documentation standards as a component of this.

  • Document all "entry points"
    • Proto messages, public keeper methods, hooks that we listen onto
    • Review the logic for these, add comments for things took time to grok
    • Ensure functionality that seems relevant is covered in tests
    • Consider making public methods private / changing or reworking API's to be more clear

Modules

@stackman27
Copy link
Contributor

Hi @ValarDragon can i do internal auditing for txfees?

Repository owner moved this from Needs Review 🔍 to Done ✅ in Osmosis Chain Development Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants