Skip to content

go/batch-submitter: Add releases for go BSS#2060

Merged
mslipper merged 3 commits intoethereum-optimism:developfrom
mslipper:feat/go-bss-releases
Jan 25, 2022
Merged

go/batch-submitter: Add releases for go BSS#2060
mslipper merged 3 commits intoethereum-optimism:developfrom
mslipper:feat/go-bss-releases

Conversation

@mslipper
Copy link
Copy Markdown
Contributor

No description provided.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jan 24, 2022

🦋 Changeset detected

Latest commit: 7612d91

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/batch-submitter-service Minor

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 2-reviewers M-ci Meta: ci related work labels Jan 24, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 24, 2022

Codecov Report

Merging #2060 (7612d91) into develop (18a8335) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2060   +/-   ##
========================================
  Coverage    74.73%   74.73%           
========================================
  Files           80       80           
  Lines         2561     2561           
  Branches       403      403           
========================================
  Hits          1914     1914           
  Misses         647      647           
Flag Coverage Δ
batch-submitter 62.50% <ø> (ø)
contracts 90.48% <ø> (ø)
core-utils 57.73% <ø> (ø)
data-transport-layer 38.64% <ø> (ø)
message-relayer 70.86% <ø> (ø)
sdk 89.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18a8335...7612d91. Read the comment docs.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This file has typos in it, its referring to canary-publish which will not exist, it should be needs.release.outputs.batch-submitter-service

Copy link
Copy Markdown
Contributor

@tynes tynes left a comment

Choose a reason for hiding this comment

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

A very common error is found in this PR - the same bug is introduced 1 in 3 times a service is added to the build pipeline. This indicates to me that this should be refactored eventually to deduplicate some of the config

Copy link
Copy Markdown
Contributor

@cfromknecht cfromknecht left a comment

Choose a reason for hiding this comment

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

LGTM barring mark's comments 👍

@mslipper mslipper force-pushed the feat/go-bss-releases branch from 648149c to 8986580 Compare January 24, 2022 23:22
@mslipper mslipper merged commit f4903ad into ethereum-optimism:develop Jan 25, 2022
theochap pushed a commit that referenced this pull request Dec 10, 2025
### Description

Implements IP Address blocking.

Closes #1566
Closes #1569
Closes #1568

---------

Co-authored-by: clabby <ben@clab.by>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-ci Meta: ci related work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants