Skip to content

feat(p2p) - more commitments#454

Merged
richard-ramos merged 2 commits into
masterfrom
feat-p2p-tls
May 11, 2026
Merged

feat(p2p) - more commitments#454
richard-ramos merged 2 commits into
masterfrom
feat-p2p-tls

Conversation

@richard-ramos
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new 2026q2 P2P/IFT commitment page for migrating nim-libp2p from BearSSL toward BoringSSL, and links it from the P2P index.

Changes:

  • Added a new commitment document: “nim-libp2p BoringSSL migration” with task breakdown and deliverables.
  • Added the commitment entry to content/p2p/index.md under 2026q2 ift:.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
content/p2p/index.md Adds a new 2026q2 commitment link under the P2P IFT section.
content/p2p/ift/2026q2-nimlibp2p-boringssl-migration.md Introduces a new commitment page describing the BoringSSL migration scope and tasks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread content/p2p/index.md Outdated

### `ift:`
* [[p2p/ift/2026q2-anoncomms-consulting|anoncomms-consulting]]
* [[p2p/ift/2026q2-nimlibp2p-boringssl|nimlibp2p-boringssl-migration]]

---

`ift-ts:p2p:ift:2026q2-nimlibp2p-boringssl`
Comment on lines +36 to +40
* fully qualified name: `ift-ts:p2p:ift:2026q2-nimlibp2p-boringssl:tls-backend-abstraction`
* owner: richard
* status: 40%
* start-date: 2026/05/03
* end-date: 2026/06/30
Comment on lines +58 to +62
* fully qualified name: `ift-ts:p2p:ift:2026q2-nimlibp2p-boringssl:crypto-rng-migration`
* owner: richard
* status: not started
* start-date: 2026/05/01
* end-date: 2026/06/30
Copy link
Copy Markdown
Collaborator

@fbarbu15 fbarbu15 left a comment

Choose a reason for hiding this comment

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

Besides what Copilot and the Validator (they are pretty much the same things) catched LGTM

@richard-ramos
Copy link
Copy Markdown
Member Author

@copilot, fix all of the observations on this PR

Agent-Logs-Url: https://github.com/vacp2p/roadmap/sessions/4cb527e4-6f1d-4cc8-8d6c-91cd322ddee9

Co-authored-by: richard-ramos <1106587+richard-ramos@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI commented May 11, 2026

@copilot, fix all of the observations on this PR

Addressed in af38131. The P2P index link now matches the file stem, the commitment identifier and task fully qualified names include -migration, and the task status now uses the validator format. I also re-ran the roadmap validator on the touched files successfully.

@richard-ramos richard-ramos merged commit b7ec197 into master May 11, 2026
2 checks passed
@richard-ramos richard-ramos deleted the feat-p2p-tls branch May 11, 2026 14:48
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