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.

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

Version Packages #485

Merged
merged 1 commit into from
Dec 11, 2024
Merged

Version Packages #485

merged 1 commit into from
Dec 11, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 11, 2024 •
edited by pr-codex bot
Loading

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.

Releases

@celo/[email protected]

Major Changes

Minor Changes

  • #420 fb08485 Thanks @shazarre! - Adds support for safe integration for L2 hotfix security council approvals

  • #429 5b02036 Thanks @shazarre! - Removes L2 BLS keys support for account:authorize

  • #447 7bc05c2 Thanks @shazarre! - validator:signed-blocks not supported on L2 anymore, fixes validator:status L2

  • #471 26b9779 Thanks @aaronmgdr! - For account:new command add alias "celoLegacy" to the --derivationPath flag

  • #459 0e559c7 Thanks @shazarre! - Introduces a new command account:claim-rpc-url allowing to claim RPC URLs

  • 76045eb Thanks @shazarre! - BLS keys are now optional as being deprecated on L2, validator:register and releasecelo:authorize no longer require them in L2 context

Patch Changes

@celo/[email protected]

Major Changes

Minor Changes

@celo/[email protected]

Major Changes

  • #340 33ad4aa Thanks @aaronmgdr! - Removes all exports under the lib/identity folder. These have been move to a new @celo/metadata-claims package and should be imported from there.

    Note that folder structure is also flattened slightly. so replace @celo/contractkit/lib/identity/claims/ with @celo/metadata-claims/lib/

    example

    - import { createAccountClaim } from '@celo/contractkit/lib/identity/claims/account'
+ import { createAccountClaim } from '@celo/metadata-claims/lib/account'
    - import { ContractKit, IdentityMetadataWrapper, newKitFromWeb3 } from '@celo/contractkit'
- import { ClaimTypes } from '@celo/contractkit/lib/identity'
+ import { ContractKit, newKitFromWeb3 } from '@celo/contractkit'
+ import { ClaimTypes, IdentityMetadataWrapper } from '@celo/metadata-claims'

    Note that Contractkit is Not a dependency. Instead when using IdentityMetadataWrapper you should make an object that satisfis the AccountMetadataSignerGetters type

    import { AccountMetadataSignerGetters } from '@celo/metadata-claims/lib/types'

    using viem it would be like

    const accountsMetaDataSignerGetters: AccountMetadataSignerGetters = {
  isAccount: async (address: string) => accounts.read.isAccount([address as Address]),
  getValidatorSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
  getVoteSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
  getAttestationSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
}

Minor Changes

  • #447 7bc05c2 Thanks @shazarre! - Exposes EpochManager.getElectedSigners contract method

  • 76045eb Thanks @shazarre! - ValidatorsWrapper: add registerValidatorNoBls to allow registration without BLS keys which are not supported in L2

Patch Changes

  • #463 eba89a3 Thanks @aaronmgdr! - Add GovernanceSlasher to RegisteredContractsEnum

  • #467 43e8474 Thanks @shazarre! - fix: add transferOwnership() to proxy abi list

  • #446 42d091f Thanks @aaronmgdr! - Bump @celo/abis-12

  • #480 b83d8c4 Thanks @aaronmgdr! - Recommended node version is now node 20

  • #455 36c4369 Thanks @aaronmgdr! - Mark contract wrapper methods that will not work on L2 because solidity contracts have onlyL1 modifier as deprecated.

    Deprecated Contract / Method Replacement or none
    Validators#registerValidator Validators#registerValidatorNoBLS
    BlockchainParams#getEpochNumberOfBlock EpochManager#getEpochNumberOfBlock
    BlockchainParams#getFirstBlockNumberForEpoch EpochManager#getFirstBlockAtEpoch
    Election#getCurrentValidatorSigners EpochManager#getElectedSigners
    Election#getGroupEpochRewards Election#getGroupEpochRewardsBasedOnScore
    GovernanceSlasher#slash GovernanceSlasher#slashL2
    DoubleSigningSlasher X
    DowntimeSlasher X

  • #400 38fe4d0 Thanks @shazarre! - Renames getElected and its usages to getElectedAccounts for EpochManagerWrapper

  • #456 d5c9204 Thanks @nicolasbrugneaux! - Backwards compat for some methods using epoch's block numbers

  • Updated dependencies [c4b9c6d, d988d31, 4ef76eb, 26b9779, fb08485]:

@celo/[email protected]

Major Changes

  • #340 33ad4aa Thanks @aaronmgdr! - Introducing @celo/metadata-claims These are a series of functions extracted from @celo/contractkit since they didnt strictly need depend on contractkit itsefl. Developers can now use IdentityMetadataWrapper with any js rpc library like ethers or viem or web3js without being forced to import ContractKit.

    Instead when using IdentityMetadataWrapper you should make an object that satisfis the AccountMetadataSignerGetters type

    import { AccountMetadataSignerGetters } from '@celo/metadata-claims/lib/types'

    using viem it would be like

    const accountsMetaDataSignerGetters: AccountMetadataSignerGetters = {
  isAccount: async (address: string) => accounts.read.isAccount([address as Address]),
  getValidatorSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
  getVoteSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
  getAttestationSigner: async (address: string) =>
    accounts.read.getValidatorSigner([address as Address]),
}

Minor Changes

Patch Changes

@celo/[email protected]

Major Changes

Patch Changes

@celo/[email protected]

Major Changes

Patch Changes

@celo/[email protected]

Minor Changes

  • #420 fb08485 Thanks @shazarre! - Now CeloProvider can be wrapped in EIP-1193 partially compatible object (request + args)

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

@celo/[email protected]

Patch Changes

PR-Codex overview

This PR primarily focuses on updating the versions of various @celo packages from beta versions to stable releases, along with some minor dependency updates and changelog entries for new features and fixes.

Detailed summary

  • Deleted multiple .changeset files.
  • Updated versions for @celo/base, @celo/utils, @celo/connect, @celo/contractkit, and several wallet packages from beta to stable.
  • Added changelog entries for patch updates in various packages.
  • Improved support for ledger transactions and fixed bugs related to wallet interactions.

The following files were skipped due to too many changes: yarn.lock

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@github-actions github-actions bot requested a review from a team as a code owner December 11, 2024 15:33
@socket-security Socket Security
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@celo/[email protected] None 0 0 B

🚮 Removed packages: npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected], npm/@celo/[email protected]

View full report↗︎

aaronmgdr
aaronmgdr approved these changes Dec 11, 2024
View reviewed changes
Copy link
Member

@aaronmgdr aaronmgdr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets go

@aaronmgdr aaronmgdr enabled auto-merge (squash) December 11, 2024 15:38
@codecov Codecov
Copy link

codecov bot commented Dec 11, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.44%. Comparing base (4bc9055) to head (1bcc1ab).
Report is 1 commits behind head on master.

Additional details and impacted files 
@@             Coverage Diff             @@
##           master     #485       +/-   ##
===========================================
+ Coverage   56.91%   73.44%   +16.53%     
===========================================
  Files         240      240               
  Lines       10180    10180               
  Branches     1606     1613        +7     
===========================================
+ Hits         5794     7477     +1683     
+ Misses       4180     2598     -1582     
+ Partials      206      105      -101
Components Coverage Δ
celocli 85.23% <ø> (+45.52%) ⬆️
sdk 68.94% <ø> (ø)
wallets 52.72% <ø> (ø)
viem-sdks 95.08% <ø> (ø)

@aaronmgdr aaronmgdr merged commit 2c37ec4 into master Dec 11, 2024
24 checks passed
@aaronmgdr aaronmgdr deleted the changeset-release/master branch December 11, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aaronmgdr aaronmgdr aaronmgdr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@aaronmgdr