Skip to content

Remove build warnings and pre-commit error #67

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
Merged
arthcp merged 4 commits into from
Feb 27, 2025
Merged

Remove build warnings and pre-commit error #67

arthcp merged 4 commits into from
Feb 27, 2025

Conversation

@rookmate
Copy link
Contributor

@rookmate rookmate commented Feb 27, 2025 •
edited
Loading

  • Fix build warnings
  • Fix pre-commit error on commit
  • Camelcase counter deploy scripts for consistency
  • Move Withdraw fees script to counter folder as it is counter specific

@rookmate rookmate changed the base branch from master to staging February 27, 2025 16:57
@arthcp arthcp merged commit 92aaa4e into staging Feb 27, 2025
1 check passed
@arthcp arthcp deleted the cosmetics branch February 27, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arthcp arthcp arthcp approved these changes

@tHeMaskedMan981 tHeMaskedMan981 Awaiting requested review from tHeMaskedMan981

@ameeshaagrawal ameeshaagrawal Awaiting requested review from ameeshaagrawal

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rookmate @arthcp