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

specify dependency version in import paths and organize imports by sorting #340

Merged

alex0207s merged 1 commit into master from style/format-imports

Jan 24, 2025

Contributor

alex0207s commented Jan 23, 2025

Specify the version of the openzeppelin dependency in the import path.
Organize imports by sorting them in the following order: dependencies > contracts > tests.
Remove variable initialization when the value is explicitly set to 0.


          specify dependency version in import paths and organize imports by so…

37cc0c8

…rting

alex0207s requested a review from NIC619

January 23, 2025 14:38

NIC619 approved these changes

View reviewed changes

contracts/AllowanceTarget.sol

Comment on lines +4 to +6

    
              import { IERC20 } from "@openzeppelin/contracts@v5.0.2/token/ERC20/IERC20.sol";

              import { SafeERC20 } from "@openzeppelin/contracts@v5.0.2/token/ERC20/utils/SafeERC20.sol";

              import { Pausable } from "@openzeppelin/contracts@v5.0.2/utils/Pausable.sol";

Contributor

NIC619 Jan 24, 2025

👍

alex0207s merged commit b01d556 into master

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet