Merged
Conversation
…hereum-optimism#740) Element ID is improperly capitalized, leading to links not going directly to the definition.
…sitory (ethereum-optimism#716) * Fix broken EAS.sol link to official Ethereum Attestation Service repository Replaced the outdated and broken link to EAS.sol in the predeploys documentation with the correct URL pointing to the official Ethereum Attestation Service (EAS) contract in the ethereum-attestation-service/eas-contracts repository. This ensures that readers can access the latest and authoritative implementation of the EAS contract. * Update predeploys.md
Co-authored-by: Emilia Hane <elsaemiliaevahane@gmail.com>
* fix typos * Update alt-da.md
…ckID in `syncStatus` method (ethereum-optimism#739) * fix(supervisor/rpc): clarify null representation for BlockRef and BlockID in syncStatus method * lintfix
Small improvements to the portal specs for things that were in the code but weren't described in the specs.
…eum-optimism#744) * feat: resetPreInterop method added to trhe managed node spec * lint added
…m#685) Link supervisor RPC error codes to contents section. Ref paradigmxyz/reth#15944 (comment)
* jovian: minimum base fee specs * Remove some repetition * Add ConfigUpdate docs * Add pseudocode * Add design doc link * Avoid respecifying extraData fields from Holocene * Fix lint errors * Run doctoc * Switch the format of the minimum base from log2 to significand + exponent * Remove the check for a blank minBaseFeeFactors * Specify the values that can be encoded by minBaseFeeFactors * Add rationale for putting the min base fee in the block header * Remove repetition * Reword * Add significand to the dictionary * Add notes about scientific notation * Use a plain u64 for min base fee
Ports `localSafe` method from Go implementation https://github.com/ethereum-optimism/optimism/blob/develop/op-supervisor/supervisor/frontend/frontend.go#L42
* chore: update words list with new entries and remove duplicates * feat: add Flashblocks to specs sidebar and update TOC formatting - Add Flashblocks entry to SUMMARY.md for sidebar navigation - Update TOC formatting in flashblocks.md to match doctoc standards * fix: add missing words to spelling dictionary - Add doctoc/DOCTOC for table of contents generation - Add technical terms: struct, structs, protobuf, pubkey - Add Brotli compression algorithm name - Add ENDIANNESS and disincentivized Fixes CircleCI spelling check failures
* feat: u16a spec Proposed specification for U16a. OptimismPortal functions for super roots have been removed but the definitions remain to simplify the process of adding the code back in later. Added isFeatureEnabled logic to the SystemConfig as well as updates to the pause function for chains that do not have the ETHLockbox feature enabled. * fix: lint * fix: use split spec format * fix: typo
…vian (ethereum-optimism#756) * explicitly require PayloadAttributes.MinBaseFee to be NULL pre jovian Closes ethereum-optimism#755 * fix typo
…nt (ethereum-optimism#763) * remove repetition in specification of SystemConfig.ConfigUpdate event The old "name" field is not part of the protocol specification, it is an implementation detail. We don't need to repeat all of the paramters each time we add a new set. The parameters should have more meaningful names. * typo * line length * just lint-specs-toc
Co-authored-by: George Knee <georgeknee@googlemail.com> Co-authored-by: Axel Kingsley <axel@oplabs.co> Co-authored-by: Teddy Knox <teddy@oplabs.co>
* fix jovian specs We had generated the upgrade transactions with an account which had already been used cast nonce 0x4210000000000000000000000000000000000004 --rpc-url=https://mainnet.optimism.io 1 This means the upgrade transactions would revert. There is already a contract at the address we were trying to deploy to. * commit script to generate specs This anticipates ethereum-optimism#673 * lint * remove file * whitespace nits
* Add script for generating upgrade txs * scripts: use uv instead of pip; create venv with uv and install jinja2 via uv\npython: update missing dependency hint to use 'uv pip install' * Update on geoknee's review feedback
* factor out "constants file" * fix bug * trim whitespace from template * regenerate Jovian specs * print a newline * tweak * tweaks * Update scripts/README.md Co-authored-by: Sebastian Stammler <seb@oplabs.co> * nicer formatting --------- Co-authored-by: Sebastian Stammler <seb@oplabs.co>
* Revert "Flashblock specs (ethereum-optimism#754)" This reverts commit 82216a7. * update authors * Apply suggestions from code review Co-authored-by: Ethnical <Ethnicalnightmare@gmail.com> * Update specs/protocol/flashblocks.md Co-authored-by: Ethnical <Ethnicalnightmare@gmail.com> * Apply suggestion from @SozinM Co-authored-by: Solar Mithril <mikawamp@gmail.com> * Apply suggestion from @SozinM Co-authored-by: Solar Mithril <mikawamp@gmail.com> * Apply suggestion from @SozinM Co-authored-by: Solar Mithril <mikawamp@gmail.com> * Apply suggestion from @SozinM Co-authored-by: Solar Mithril <mikawamp@gmail.com> * Apply suggestion from @SozinM Co-authored-by: Solar Mithril <mikawamp@gmail.com> * fix: resolve markdown linting issues in flashblocks.md - Fix line length violations (MD013) - Remove trailing spaces (MD009) - Convert hard tabs to spaces (MD010) - Fix list marker spacing (MD030) - Reduce lint errors from 176 to 28 * fix: resolve additional markdown linting issues in flashblocks.md - Fix unordered list indentation (MD007) - Remove multiple consecutive blank lines (MD012) - Convert bare URLs to markdown links (MD034) - Fix heading hierarchy (MD001) - Reduce lint errors from 23 to 18 * turn off remaining lints because i investigated them and they're overly cautious errors * Update flashblocks.md Co-authored-by: Blessing Krofegha <bkrofegha@gmail.com> * Update flashblocks.md Co-authored-by: Blessing Krofegha <bkrofegha@gmail.com> * fix: add flashblocks to SUMMARY.md and update words.txt - Add Flashblocks entry to SUMMARY.md for sidebar navigation - Add technical terms to words.txt: Brotli, disincentivized, doctoc, DOCTOC, ENDIANNESS, protobuf, pubkey, struct, structs - Sort and deduplicate words.txt This applies the necessary fixes from PR ethereum-optimism#754 while preserving the original authorship of dmarz's commits. * fix: convert [-*+] bullet points to proper markdown syntax - Replace [-*+] with standard markdown bullet points (-) - Fixes markdown linting warnings about potential incomplete links - Improves readability and compliance with markdown standards * fix: correct malformed markdown links - Convert reference-style links with inline URLs to proper inline links - Remove malformed [text][reference](url) syntax - Fix links to ethereum-optimism repository references - Remove extra asterisks from markdown formatting - Resolves markdown linting warnings about incomplete link references * fix: resolve markdown linting errors - Add top-level heading '# Flashblocks' to fix MD041 first-line-heading rule - Fix MD029 ordered list prefix issues by properly indenting bullet points under numbered items - Ensure sequential numbering (1, 2, 3, 4, 5, 6, 7) for all ordered lists - Indent all sub-bullet points with 3 spaces to be properly nested under their parent items * fix: resolve remaining markdown linting errors - Fix MD051 link fragment issues by updating TOC links to use correct fragment IDs - Fix MD059 descriptive link text by replacing '[here]' with '[flashblocks-websocket-proxy repository]' - All markdown linting errors now resolved (0 errors) * fix: update TOC format to match doctoc expectations - Remove authors comment section that was causing conflicts - Update TOC entries to use bold formatting for data structure headings - Remove manual 'Table of Contents' header to match doctoc format * fix: update words.txt with new terms and corrections - Add new technical terms including Airgap, Basefee, Bootnode, and others - Remove outdated or duplicate entries to streamline the list - Ensure consistent casing and formatting for terms throughout the file * fix: add missing technical terms to spelling dictionary - Add doctoc/DOCTOC for table of contents generation - Add technical terms: struct, structs, protobuf, pubkey - Add Brotli compression algorithm name - Add ENDIANNESS and disincentivized - Sort and deduplicate words.txt for consistency Fixes spelling check failures introduced by doctoc updates --------- Co-authored-by: dmarzzz <danmarzec@icloud.com> Co-authored-by: Ethnical <Ethnicalnightmare@gmail.com> Co-authored-by: Solar Mithril <mikawamp@gmail.com>
…m-optimism#788) Co-authored-by: George Knee <george@oplabs.co>
…in activation block (ethereum-optimism#789) * jovian, isthmus: be more precise about l1 block attributes tx choice in activation block * isthmus: harmonize layout of specs with ecotone and jovian * lint * fix SUMMARY.md
* clarify specs for Fusaka defense * respond to review * lint
* fix!: multiply operatorFeeScalar by 100 instead of dividing by 1e6 * chore: move under jovian specs * fix: handle feedbacks * feat: add upgrade tx specs * fix: typo * fix: lints * lint --------- Co-authored-by: leruaa <aurelien.catinon@gmail.com> Co-authored-by: geoknee <georgeknee@googlemail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.