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

Correct typo module imports for just test CLI and unit fix #181 #182

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

s3bc40
Copy link
Contributor

@s3bc40 s3bc40 commented Jan 17, 2025

Related to: #181
OS: Ubuntu24
Mox version: Moccasin CLI v0.3.6


Just test command failed due to wrong typo on Vyper module import PatrickAlphaC.test_repo

-> Fixed import in Vyper related file

from patrickalphac.test_repo import my_contract

Found other import failure on module Auth imported like auth initially:

# tests/data/complex_project/contracts/InitializedAuth.vy
import auth

-> Fixed import by using aliases

import Auth as auth

… cli unit

- just test command failed due to wrong typo on vyper module import PatrickAlphaC.test_repo.
    - fixed import in vypers related files
- found other import failure on module Auth imported like auth initially
    - fixed import by using aliases
@PatrickAlphaC PatrickAlphaC merged commit 0acd37f into Cyfrin:main Jan 19, 2025
2 checks passed
@PatrickAlphaC
Copy link
Member

Thanks!! These are very helpful!

@s3bc40
Copy link
Contributor Author

s3bc40 commented Jan 19, 2025

Happy to contribute and help! Thanks for the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants