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

#846: change minimum version of CryptoSwift dependency #848

Open
wants to merge 21 commits into
base: develop
Choose a base branch
from

Conversation

g1umov
Copy link

@g1umov g1umov commented Feb 10, 2024

The minimum version of 1.5.1 is too small. These changes resolve dependency conflicts that have the 'CryptoSwift' dependency within the SPM package, e.g. 'SwiftLint' that uses the 'CryptoSwift' too

Summary of Changes

Fixes # (if applicable - add the number of issue this PR addresses)

Test Data or Screenshots

By submitting this pull request, you are confirming the following:
  • I have reviewed the Contribution Guidelines.
  • I have performed a self-review of my own code.
  • I have updated my repository to match the develop branch.
  • I have included test data or screenshots that prove my fix is effective or that my feature works.
  • I have checked that all tests work and swiftlint is not throwing any errors/warnings.

albertopeam and others added 21 commits January 22, 2023 11:18
…-callContractMethod

Renamed callContractMethod to call for ReadOperation
…op-v4.0-up-to-date

Chore/develop v4.0 up to date
Refactor createNewAccount function to be an internal function and remove unused password argument
* develop: (24 commits)
  refactor: random bytes generation; data and string extensions update (web3swift-team#791)
  encodeTopic bytes should accept hex string
  add more test for encodeTopic
  Trim Trailing Whitespace
  remove a force unwrap
  feat: enable encode Event and IETH.getLogs
  - isHardened check removed from createNewCustomChildAccount
  - linter warning fixed
  chore: trailing closure style for compactMap
  fix: used short notation for null checking
  fix: trimming / only if it's present as a prefix; using let for 'newPath';
  - Test testAddressGeneration modified with hardcoded addresses
  chore: refactoring of createNewCustomChildAccount
  - bug with original pathAppendix fixed
  fix: createNewAccount always non-hardened derivation
  - Trailing Whitespace fixed in tests
  - not used code removed - small refactoring
  - closure style updated - pathAppendix declaration place changed
  - linter Trailing Whitespace Violation fixed
  - Grammar fixed - Removed prefix path from address generation
  ...
* develop: (24 commits)
  chore: added docs; new test for ABI.Element.Function;
  chore: Update Sources/Web3Core/EthereumNetwork/Request/APIRequest+Methods.swift
  chore: Update Sources/Web3Core/EthereumNetwork/Request/APIRequest+Methods.swift
  fix: testDecodeMulticallCopy
  fix: code spell
  chore: error messages updated
  chore: isHex string extension tests
  fix: isHex string extension function refactoring
  chore: avoiding try? in tests (try is more preffered)
  chore: missing space added
  chore: comment refactoring
  chore: docs refactoring
  chore: docs refactoring
  chore: code reordering
  chore: error message update
  update function documentation
  fix codespell
  resolve reviews suggestions
  add missing test contract bytecode
  fix eth method
  ...

# Conflicts:
#	Tests/web3swiftTests/localTests/AdvancedABIv2Tests.swift
#	Tests/web3swiftTests/localTests/UncategorizedTests.swift
* chore: removed CocoaPods

* chore: removing cocoapods support

* chore: fixed linked to workflow file

Signed-off-by: Jenea Vranceanu <[email protected]>

* chore: swift badge colour change

Signed-off-by: Jenea Vranceanu <[email protected]>

* chore: added more examples to README.md

Signed-off-by: Jenea Vranceanu <[email protected]>

* chore: rephrasing, fixed typos README.md

Signed-off-by: Jenea Vranceanu <[email protected]>

* chore: Update README.md

Signed-off-by: Jenea Vranceanu <[email protected]>

* chore: hiding support badge in README.md

Signed-off-by: Jenea Vranceanu <[email protected]>

---------

Signed-off-by: Jenea Vranceanu <[email protected]>
The minimum version of 1.5.1 is too small. These changes resolve dependency conflicts that have the 'CryptoSwift' dependency within the SPM package
@JeneaVranceanu JeneaVranceanu self-requested a review February 11, 2024 20:13
@yaroslavyaroslav
Copy link
Collaborator

Could you please point it to develop instead?

@g1umov g1umov changed the base branch from develop-4.0 to develop December 29, 2024 16:06
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.

5 participants