refactor: do not keep track of membership registration order #133
richard-ramosAnnotations
14 warnings
|
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint:
script/Deploy.s.sol#L9
imported name UUPSUpgradeable is not used
|
|
lint:
script/Deploy.s.sol#L43
Avoid to use inline assembly. It is acceptable only in rare cases
|
|
lint:
script/Deploy.s.sol#L48
Avoid to use inline assembly. It is acceptable only in rare cases
|
|
lint:
src/Membership.sol#L150
Provide an error message for require
|
|
lint:
src/Membership.sol#L151
Provide an error message for require
|
|
lint:
src/Membership.sol#L152
Provide an error message for require
|
|
lint:
src/Membership.sol#L153
Provide an error message for require
|
|
lint:
src/WakuRlnV2.sol#L32
Variable name must be in mixedCase
|
|
lint:
src/WakuRlnV2.sol#L50
Error message for require is too long
|
|
lint:
src/WakuRlnV2.sol#L289
Provide an error message for require
|
|
test
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
coverage
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, pnpm/action-setup@v2, actions/setup-node@v3, codecov/codecov-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|