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

Custom errors, etc #619

Merged
merged 16 commits into from
Feb 26, 2024
Merged

Custom errors, etc #619

merged 16 commits into from
Feb 26, 2024

Conversation

kumaryash90
Copy link
Member

No description provided.

@@ -74,7 +84,10 @@
function getAllPublishedContracts(
address _publisher
) external view returns (CustomContractInstance[] memory published) {
CustomContractInstance[] memory linkedData = prevPublisher.getAllPublishedContracts(_publisher);
CustomContractInstance[] memory linkedData;

Check warning

Code scanning / Slither

Uninitialized local variables Medium

@@ -202,7 +225,10 @@
function getPublishedUriFromCompilerUri(
string memory compilerMetadataUri
) public view returns (string[] memory publishedMetadataUris) {
string[] memory linkedUris = prevPublisher.getPublishedUriFromCompilerUri(compilerMetadataUri);
string[] memory linkedUris;

Check warning

Code scanning / Slither

Uninitialized local variables Medium

_publisher,
_contractId
);
CustomContractInstance[] memory linkedVersions;

Check warning

Code scanning / Slither

Uninitialized local variables Medium

_setupRole(DEFAULT_ADMIN_ROLE, _msgSender());
constructor(
address _defaultAdmin,
address _trustedForwarder,

Check notice

Code scanning / Slither

Local variable shadowing Low

Comment on lines +32 to +39
function _efficientHash(bytes32 a, bytes32 b) private pure returns (bytes32 value) {
/// @solidity memory-safe-assembly
assembly {
mstore(0x00, a)
mstore(0x20, b)
value := keccak256(0x00, 0x40)
}
}

Check warning

Code scanning / Slither

Assembly usage Warning

Copy link

codecov bot commented Feb 23, 2024

Codecov Report

Attention: Patch coverage is 77.35849% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 64.73%. Comparing base (effd208) to head (83e83fd).

Files Patch % Lines
contracts/extension/Drop1155.sol 57.14% 3 Missing ⚠️
contracts/extension/DropSinglePhase.sol 50.00% 3 Missing ⚠️
contracts/extension/PlatformFee.sol 40.00% 3 Missing ⚠️
...acts/legacy-contracts/extension/PlatformFee_V1.sol 25.00% 3 Missing ⚠️
contracts/lib/MerkleProof.sol 0.00% 3 Missing ⚠️
contracts/extension/SignatureMintERC721.sol 66.66% 2 Missing ⚠️
...racts/extension/SignatureMintERC721Upgradeable.sol 66.66% 2 Missing ⚠️
contracts/prebuilts/pack/Pack.sol 80.00% 2 Missing ⚠️
contracts/extension/PrimarySale.sol 50.00% 1 Missing ⚠️
...acts/legacy-contracts/extension/PrimarySale_V1.sol 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #619      +/-   ##
==========================================
+ Coverage   64.55%   64.73%   +0.17%     
==========================================
  Files         216      216              
  Lines        6701     6700       -1     
==========================================
+ Hits         4326     4337      +11     
+ Misses       2375     2363      -12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kumaryash90 kumaryash90 merged commit 8959088 into main Feb 26, 2024
9 checks passed
@kumaryash90 kumaryash90 deleted the yash/fix-size branch February 26, 2024 11:52
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.

1 participant