Skip to content

Comments

integration-tests: Add tests for system addrs on verifiers/replicas#2478

Merged
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
mslipper:feat/replica-system-addr-tests
Apr 18, 2022
Merged

integration-tests: Add tests for system addrs on verifiers/replicas#2478
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
mslipper:feat/replica-system-addr-tests

Conversation

@mslipper
Copy link
Collaborator

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Apr 18, 2022

🦋 Changeset detected

Latest commit: 8a49ec8

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/integration-tests 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 the A-integration Area: integration tests label Apr 18, 2022
@mergify mergify bot requested review from cfromknecht and tuxcanfly April 18, 2022 11:39
@mslipper mslipper force-pushed the feat/replica-system-addr-tests branch 3 times, most recently from 4246c8f to febc309 Compare April 18, 2022 12:06
@mslipper mslipper force-pushed the feat/replica-system-addr-tests branch from febc309 to 3126126 Compare April 18, 2022 12:41
Copy link
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

@mergify
Copy link
Contributor

mergify bot commented Apr 18, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Apr 18, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit db02f97 into ethereum-optimism:develop Apr 18, 2022
nebojsa94 pushed a commit to Tenderly/optimism that referenced this pull request Apr 26, 2022
…thereum-optimism#2478)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
@mslipper mslipper mentioned this pull request Apr 28, 2022
theochap pushed a commit that referenced this pull request Dec 10, 2025
## Problem

The `README.md` file in the `kona-peers` crate had improper markdown
formatting that made it difficult to read and inconsistent with other
crates in the repository:

- Malformed title starting with `1.#` instead of proper markdown header
- Content was written as Rust doc comments (`//!`) rather than markdown
- Missing standard badge links present in other crate READMEs
- Overall inconsistent formatting compared to repository standards

## Solution

Fixed the markdown formatting to match the consistent pattern used
across other crates in the repository:

1. **Fixed title**: Changed `1.#` to proper markdown header `#
\`kona-peers\``
2. **Added standard badges**: Added CI, crates.io, license, and codecov
badge links following the same pattern as other crates
3. **Converted doc comments**: Removed Rust doc comment syntax (`//!`)
and converted content to proper markdown paragraphs
4. **Preserved technical content**: Maintained all the comprehensive
documentation about networking utilities, node records, peer IDs, and
ENRs

## Before
```markdown
1.#



//! Networking Utilities ported from reth.
//!
//! Much of this module is ported from
//! <https://github.com/paradigmxyz/reth/blob/...>.
//!
//! This module manages and converts Ethereum network entities...
```

## After
```markdown
# `kona-peers`

<a href="https://github.com/op-rs/kona/actions/workflows/rust_ci.yaml">...</a>
<a href="https://crates.io/crates/kona-peers">...</a>
<a href="https://github.com/op-rs/kona/blob/main/LICENSE.md">...</a>
<a href="https://img.shields.io/codecov/c/github/op-rs/kona">...</a>

Networking Utilities ported from reth.

Much of this module is ported from
<https://github.com/paradigmxyz/reth/blob/...>.

This module manages and converts Ethereum network entities...
```

The README now properly renders as markdown and follows the consistent
formatting pattern established by other crates in the repository, while
preserving all the valuable technical documentation about the different
node record types (`PeerId`, `NodeRecord`, `discv5::Enr`, `AnyNode`).

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 Share your feedback on Copilot coding agent for the chance to win a
$200 gift card! Click
[here](https://survey.alchemer.com/s3/8343779/Copilot-Coding-agent) to
start the survey.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: refcell <21288394+refcell@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-integration Area: integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants