Skip to content

Extended test coverage of core-utils#1422

Merged
tynes merged 1 commit intoethereum-optimism:developfrom
rajivpo:rajiv/increase-coverage-core-utils
Sep 7, 2021
Merged

Extended test coverage of core-utils#1422
tynes merged 1 commit intoethereum-optimism:developfrom
rajivpo:rajiv/increase-coverage-core-utils

Conversation

@rajivpo
Copy link
Copy Markdown
Contributor

@rajivpo rajivpo commented Sep 5, 2021

Description

  • Increase coverage in core-utils package (71.3% up from 64%, 100% for all files excluding l2context.ts and watcher.ts)
  • This doesn't currently handler increased coverage around l2context and watcher but that can be included if preferred.

Additional context
This shouldn't be merged until #1412 is merged.

Metadata

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 5, 2021

🦋 Changeset detected

Latest commit: eb0854e

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

This PR includes changesets to release 1 package
Name Type
@eth-optimism/core-utils 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

@github-actions github-actions bot added A-ts-packages A-pkg-core-utils Area: packages/core-utils labels Sep 5, 2021
@rajivpo rajivpo marked this pull request as ready for review September 7, 2021 17:08
@tynes tynes merged commit 1a73fac into ethereum-optimism:develop Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-pkg-core-utils Area: packages/core-utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants