Skip to content

chore: updates more rfcs#177

Merged
SWvheerden merged 1 commit into
mainfrom
sw_updates
Jun 29, 2026
Merged

chore: updates more rfcs#177
SWvheerden merged 1 commit into
mainfrom
sw_updates

Conversation

@SWvheerden

@SWvheerden SWvheerden commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Description

Removes maintainer field as many of the listed maintainers are not with Tari anymore,
Fixes up wrong and missing fields.
Does not update to tips yet

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates several Tari RFC documents, notably removing maintainer metadata, renaming 'Tari' to 'MinoTari' in wallet contexts, updating deep link conventions, removing the Store and Forward (SAF) protocol, and expanding the mining proof-of-work specification to four algorithms with adjusted target times. The review feedback highlights a mathematical contradiction in the mining target block times (which should be 200 seconds instead of 120 seconds), suggests a minor grammatical correction ('an RxM'), and points out leftover SAF references in the peer-to-peer messaging protocol that need to be cleaned up for consistency.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/RFC-0131_Mining.md
Comment thread src/RFC-0131_Mining.md
Comment thread src/RFC-0172_PeerToPeerMessagingProtocol.md
@SWvheerden SWvheerden changed the title upadates more rfcs chore: updates more rfcs Jun 19, 2026
@SWvheerden SWvheerden merged commit 277f190 into main Jun 29, 2026
3 of 6 checks passed
@SWvheerden SWvheerden deleted the sw_updates branch June 29, 2026 11:54
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.

1 participant