feat: latest solidity version & remove license header #649
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
MiguelLZPF
force-pushed
the
feat/BBND-1057-latest-solidity-version-remove-license-header
branch
2 times, most recently
from
c9adecd to
363e817
Compare
MiguelLZPF
force-pushed
the
feat/BBND-1057-latest-solidity-version-remove-license-header
branch
from
363e817 to
502d554
Compare
MiguelLZPF
added
the
Improvement
MiguelLZPF
force-pushed
the
feat/BBND-1057-latest-solidity-version-remove-license-header
branch
2 times, most recently
from
1562899 to
7a10895
Compare
Axel-IoBuilders
force-pushed
the
develop
branch
from
16c8659 to
8c532ff
Compare
Upgrade all Solidity contracts from fixed version pragma solidity 0.8.18; to flexible version range pragma solidity >=0.8.0 <0.9.0; This enables compatibility with newer Solidity versions while maintaining compatibility with existing 0.8.18 compiled contracts. Updated 315+ contract files across all layers: - Layer 0: Core storage wrappers and libraries - Layer 1: Business logic implementations - Layer 2: Asset-specific functionality - Layer 3: Jurisdiction-specific implementations - Factory and resolver contracts - Mock contracts for testing - Test utilities and time travel contracts Also updated .solhint.json compiler-version rule to accept the new flexible version range. Signed-off-by: Miguel_LZPF <[email protected]>
Update Hardhat configuration and dependencies: - Upgrade primary compiler from 0.8.18 to 0.8.28 - Update EVM version from 'london' to 'cancun' - Update hardhat network hardfork to 'cancun' - Remove redundant global compiler settings - Clean up dependency compiler paths Update development dependencies: - Upgrade @typechain/ethers-v5 from ^10.1.0 to ^11.1.2 - Upgrade hardhat from ^2.22.19 to ^v2.26.3 - Add TypeChain override for consistent version resolution - Remove unused blob dependency - Keep publishConfig for proper scoped package publishing Signed-off-by: Miguel_LZPF <[email protected]>
Remove large Apache License 2.0 headers from all Solidity files and replace with proper SPDX identifiers for consistency with industry standards and reduced file size. Changes: - Updated 315 Solidity files to use SPDX-License-Identifier: Apache-2.0 - Removed verbose Apache license text blocks - Fixed TypeScript import for IIdFactory__factory in tasks/deploy.ts - Maintained license compliance while improving code readability Signed-off-by: Miguel_LZPF <[email protected]>
MiguelLZPF
force-pushed
the
feat/BBND-1057-latest-solidity-version-remove-license-header
branch
from
7a10895 to
21d0193
Compare
- Change pragma from exact version 0.8.18 to range >=0.8.0 <0.9.0 - Remove full Apache license headers, keeping only SPDX identifiers - Standardize license to Apache-2.0 for consistency - Fix SPDX license header ordering (license first, then pragma) - Resolves build failures after develop rebase Files updated: - IBeneficiaries.sol - BeneficiariesStorageWrapper.sol - Beneficiaries.sol - BeneficiariesFacet.sol - BeneficiariesFacetTimeTravel.sol All builds and tests pass successfully. Signed-off-by: Miguel_LZPF <[email protected]>
AlbertoMolinaIoBuilders
approved these changes
Sep 18, 2025
Axel-IoBuilders
approved these changes
Sep 18, 2025
MiguelLZPF
deleted the
feat/BBND-1057-latest-solidity-version-remove-license-header
branch
themariofrancia
pushed a commit
that referenced
this pull request
Sep 25, 2025
Signed-off-by: Miguel_LZPF <[email protected]> Signed-off-by: Mario Francia <[email protected]>
themariofrancia
pushed a commit
that referenced
this pull request
Dec 9, 2025
Signed-off-by: Miguel_LZPF <[email protected]>
themariofrancia
pushed a commit
that referenced
this pull request
Dec 10, 2025
Signed-off-by: Miguel_LZPF <[email protected]>
themariofrancia
pushed a commit
that referenced
this pull request
Dec 10, 2025
Signed-off-by: Miguel_LZPF <[email protected]> Signed-off-by: Mario Francia <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This pull request upgrades Solidity compiler configuration and standardizes license headers across the entire smart contract codebase. The changes improve compatibility, reduce bundle size, and ensure consistent licensing practices.
Solidity Version Upgrade:
pragma solidity 0.8.18;to flexiblepragma solidity >=0.8.0 <0.9.0;License Header Standardization:
SPDX-License-Identifier: Apache-2.0as first lineDevelopment Dependencies:
Impact:
All changes maintain full backward compatibility with existing deployed contracts.
Type of change
Testing
Verification completed:
Node version:
Checklist