Skip to content

docs: remove deprecated rollup-driver glossary entry#1612

Open
wolfgang1211 wants to merge 1 commit into
base:masterfrom
wolfgang1211:fix/remove-rollup-driver-glossary-entry
Open

docs: remove deprecated rollup-driver glossary entry#1612
wolfgang1211 wants to merge 1 commit into
base:masterfrom
wolfgang1211:fix/remove-rollup-driver-glossary-entry

Conversation

@wolfgang1211

Copy link
Copy Markdown

Closes the TODO in glossary.mdx that asks to delete the ### Rollup Driver entry and replace references with "derivation process".

Changes:

  • Removed the deprecated ### Rollup Driver section from glossary.mdx
  • Updated inline references across specs to use "derivation process"
  • Removed orphaned link definitions pointing to the deleted anchor

Related TODO at:
docs/base-chain/specs/reference/glossary.mdx line ~760

Verification:

  • git diff --check
  • Custom scan confirmed no rollup-driver references remain in the changed files
  • node scripts/lint-mdx.js was run on Windows; it reports pre-existing/tooling issues caused by CRLF frontmatter detection and closing code fences being treated as missing languages.

The 'Rollup Driver' glossary entry was marked with a TODO to be deleted and replaced by 'derivation process' or 'derivation logic' throughout.

Changes:

- glossary.mdx: replace [rollup driver] inline links with 'derivation process' and remove the ### Rollup Driver section

- execution/index.mdx: replace [rollup driver][rollup-driver] link and remove orphaned link definition

- consensus/index.mdx: remove [g-rollup-driver] link definition and update prose to use 'derivation process'

- consensus/rpc.mdx: update plain-text reference for consistency
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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