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.

Sign up for GitHub

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

Jump to bottom

Cleanup Applicator #303

Closed
mvandeberg opened this issue Nov 12, 2024 · 0 comments · Fixed by #308
Closed

Cleanup Applicator #303

mvandeberg opened this issue Nov 12, 2024 · 0 comments · Fixed by #308
Assignees
@mvandeberg
Labels
enhancement New feature or request

Comments

@mvandeberg
Copy link
Member

In preparation for future work, the applicator needs to be cleaned up.

  • Improve function names to be specific to blocks/transactions
  • Applicator instance should be a, not b.
@mvandeberg mvandeberg added the enhancement New feature or request label Nov 12, 2024
@mvandeberg mvandeberg self-assigned this Nov 12, 2024
mvandeberg added a commit that referenced this issue Nov 12, 2024
@mvandeberg
#303: Rename applicator instance to 'a' and clarify block context of …
6629afc 
…functions
@mvandeberg mvandeberg mentioned this issue Nov 12, 2024
Merged
5 tasks
mvandeberg added a commit that referenced this issue Nov 12, 2024
@mvandeberg
#303: 'removeEntry' -> 'removeBlockEntry'
0d49f49
mvandeberg added a commit that referenced this issue Nov 12, 2024
@mvandeberg
#303: Increase linter timeout to 4 minutes
6b89505
mvandeberg added a commit that referenced this issue Nov 13, 2024
@mvandeberg
#303: Rename applicator instance to 'a' and clarify block context of …
8e41794 
…functions
mvandeberg added a commit that referenced this issue Nov 13, 2024
@mvandeberg
#303: 'removeEntry' -> 'removeBlockEntry'
b912d0f
mvandeberg added a commit that referenced this issue Nov 13, 2024
@mvandeberg
Merge pull request #308 from koinos/303-cleanup-applicator
5fbb56a 
#303: Cleanup applicator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mvandeberg mvandeberg

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#303: Cleanup applicator koinos/koinos-p2p
1 participant
@mvandeberg