Skip to content

Conversation

@snario
Copy link
Contributor

@snario snario commented Apr 23, 2021

There are two issues on Sentry now:

  • release: Discarded invalid value
  • exception.values.0.stacktrace.frames: Discarded invalid value

These commits address them:

  • build: use npm script to start base service based services
  • fix: remove invalid slash character from sentry release name
  • fix: set stackAttributeKey to conventional 'err' key

@changeset-bot
Copy link

changeset-bot bot commented Apr 23, 2021

🦋 Changeset detected

Latest commit: dfb9389

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@eth-optimism/batch-submitter Patch
@eth-optimism/data-transport-layer Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@snario snario removed the request for review from karlfloersch April 23, 2021 01:30
Copy link
Contributor

@annieke annieke left a comment

Choose a reason for hiding this comment

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

💯

@snario snario merged commit d1f2644 into master Apr 23, 2021
@snario snario deleted the liam/use-node branch April 23, 2021 01:52
agusduha pushed a commit that referenced this pull request Sep 25, 2025
github-merge-queue bot pushed a commit that referenced this pull request Nov 6, 2025
* feat: fee splitter system (#469)


----

Co-authored-by: Disco <[email protected]>
Co-authored-by: Flux <[email protected]>
Co-authored-by: Chiin <[email protected]>

* fix: failing tests (#553)

* refactor: revenue sharing config (#538)

* fix: pre pr and semgrep (#565)

* fix: warnings (#571)

* fix: tests failing on fork environment (#575)

* fix: use encode call on constructors (#578)

* feat: add integration splitter test (#581)

* fix: integration test nits (#583)

* test: more coverage on splitter tests (#594)


---------

Co-authored-by: Chiin <[email protected]>

* fix: add missing operator fee vault field (#616)

* feat: invariants and tests (#595)

* fix: ir informationals (#609)

* fix: pre pr (#622)

* refactor: use cdm (#624)

* fix: update withdrawal gas limit value on check (#627)

* fix: remove rev share field from deploy op chain input struct (#628)

* refactor: remove initializer and vaults getter changes (#631)

* fix: remove immutable check over vaults (#634)


Co-authored-by: 0xng <[email protected]>
Co-authored-by: Funkornaut <[email protected]>
Co-authored-by: Joxess <[email protected]>

* fix: audit findings (#658)

* fix: pre pr

* feat: add user guide docs for rev sharing on op deployer (#666)

* docs: add create2 comment (#667)

---------

Co-authored-by: Funkornaut <[email protected]>
Co-authored-by: Flux <[email protected]>
Co-authored-by: Chiin <[email protected]>
Co-authored-by: Simon Something /DrGoNoGo <[email protected]>
Co-authored-by: 0xng <[email protected]>
Co-authored-by: Joxess <[email protected]>
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.

4 participants