Skip to content
This repository was archived by the owner on Apr 18, 2025. It is now read-only.

Conversation

@lispc
Copy link

@lispc lispc commented May 28, 2024

Description

[PR description]

Issue Link

[link issue here]

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Contents

  • [item]

Rationale

[design decisions and extended information]

How Has This Been Tested?

[explanation]


How to fill a PR description

Please give a concise description of your PR.

The target readers could be future developers, reviewers, and auditors. By reading your description, they should easily understand the changes proposed in this pull request.

MUST: Reference the issue to resolve

Single responsability

Is RECOMMENDED to create single responsibility commits, but not mandatory.

Anyway, you MUST enumerate the changes in a unitary way, e.g.

This PR contains:
- Cleanup of xxxx, yyyy
- Changed xxxx to yyyy in order to bla bla
- Added xxxx function to ...
- Refactored ....

Design choices

RECOMMENDED to:

  • What types of design choices did you face?
  • What decisions you have made?
  • Any valuable information that could help reviewers to think critically

@lispc lispc changed the title feat: update latest curie l1 bytecode feat: l1 fee contract upgrade part1 May 29, 2024
@lispc lispc marked this pull request as ready for review May 29, 2024 06:28
@lispc lispc changed the title feat: l1 fee contract upgrade part1 feat: l1 fee contract upgrade part2 May 29, 2024
@lispc
Copy link
Author

lispc commented May 29, 2024

passed real traces tests

@lispc lispc merged commit e4095f7 into develop May 29, 2024
@lispc lispc deleted the tmp/logging branch May 29, 2024 09:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants