Skip to content

Conversation

jstuczyn
Copy link
Contributor

@jstuczyn jstuczyn commented Oct 8, 2025

Ticket

N/A

Description

this PR moves nym-gateway-probe (alongside nym-common and nym-connection-monitor) to the monorepo.

Checklist:

  • Changelog

Screenshots (optional, if UI related)


This change is Reviewable

@jstuczyn jstuczyn requested a review from pronebird October 8, 2025 15:20
Copy link
Collaborator

@pronebird pronebird left a comment

Choose a reason for hiding this comment

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

Why do we need to move nym-connection-monitor and nym-common to nym repo?

Reviewable status: 0 of 30 files reviewed, all discussions resolved

Copy link
Collaborator

@pronebird pronebird left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 30 files reviewed, 2 unresolved discussions


nym-vpn-core/Cargo.toml line 5 at r2 (raw file):

members = [
    "crates/nym-apple-network",
    "crates/nym-common",

Probably need to revert this too


nym-vpn-core/Cargo.toml line 6 at r2 (raw file):

    "crates/nym-apple-network",
    "crates/nym-common",
    "crates/nym-connection-monitor",

and this

Copy link
Collaborator

@pronebird pronebird left a comment

Choose a reason for hiding this comment

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

@pronebird reviewed 16 of 30 files at r1, 14 of 14 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @jstuczyn)

@jstuczyn jstuczyn force-pushed the feature/gateway-probe-migration branch from c4cd65d to 0a337a0 Compare October 15, 2025 12:56
@jstuczyn jstuczyn enabled auto-merge (squash) October 15, 2025 12:56
@jstuczyn jstuczyn merged commit 1f3defc into develop Oct 15, 2025
11 of 14 checks passed
@jstuczyn jstuczyn deleted the feature/gateway-probe-migration branch October 15, 2025 13:07
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.

2 participants