Skip to content

contracts: script for making markdown file with all our contract addresses#456

Merged
gakonst merged 4 commits intomasterfrom
feat/regenesis-generate-markdown
Apr 15, 2021
Merged

contracts: script for making markdown file with all our contract addresses#456
gakonst merged 4 commits intomasterfrom
feat/regenesis-generate-markdown

Conversation

@gigamesh
Copy link
Copy Markdown
Contributor

@gigamesh gigamesh commented Apr 13, 2021

Credit to Pacien Boisson @pakocrew

Please review carefully as this will be the source of truth for the near future

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 13, 2021

⚠️ No Changeset found

Latest commit: e32b665

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@gigamesh gigamesh force-pushed the feat/regenesis-generate-markdown branch 2 times, most recently from 121858a to 53146f5 Compare April 13, 2021 17:04
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I wasn't sure of the best place to run it but figured it should happen here

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.

Probably should be part of the deploy script.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

moved 👍

@snario snario requested a review from smartcontracts April 13, 2021 17:16
@gigamesh gigamesh force-pushed the feat/regenesis-generate-markdown branch from 63daf96 to 52e0bde Compare April 13, 2021 21:10
Copy link
Copy Markdown
Contributor

@snario snario left a comment

Choose a reason for hiding this comment

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

Instead of a column named Open could we just hyperlink the address of the contract to make the table width smaller?

@tynes
Copy link
Copy Markdown
Contributor

tynes commented Apr 14, 2021

Instead of a column named Open could we just hyperlink the address of the contract to make the table width smaller?

I'd support this

@gigamesh gigamesh force-pushed the feat/regenesis-generate-markdown branch from b76ad12 to 71f9850 Compare April 14, 2021 17:31
const chainId = Number(
fs.readFileSync(`./deployments/${deployment}/.chainId`)
);
const network = networks[chainId];
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.

Could we add some sort of per-network annotations that can be injected into the markdown? E.g., kovan-v1 is outdated and will no longer be in use. We'll want to stop showing those addresses to people once we stop using the contracts. We probably want to add some sort of text to that section to make it clear that those contracts are deprecated. Also probably want to add the option to hide a deployment completely.

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.

Let's do this in a separate PR, if you have a specific idea could you please open an issue for it?

@smartcontracts
Copy link
Copy Markdown
Contributor

Also requesting that we add commit credits for @pakocrew

@gakonst gakonst force-pushed the feat/regenesis-generate-markdown branch from 71f9850 to e32b665 Compare April 15, 2021 14:38
@gakonst
Copy link
Copy Markdown
Contributor

gakonst commented Apr 15, 2021

I rebased and git commit --amend --author="Pacien Boisson <pakokrew@gmail.com" --no-edit the first commit to credit @pakokrew

@gakonst gakonst merged commit 3c92bdb into master Apr 15, 2021
@gakonst gakonst deleted the feat/regenesis-generate-markdown branch April 15, 2021 14:40
InoMurko referenced this pull request in omgnetwork/optimism May 25, 2021
…esses (#456)

* contracts: adds generate-markdown script for making markdown file with all our contract addresses

* Moves generate-markdown to deploy script

* Adds Goerli chain Id

* Removes etherscan Open column

Co-authored-by: Pacien Boisson <pakokrew@gmail.com>
bap2pecs added a commit to babylonlabs-io/optimism that referenced this pull request Jul 31, 2024
theochap pushed a commit that referenced this pull request Dec 10, 2025
* fix: downgrade for release plz (#458)

* chore(bin): Remove `kt` (#461)

* chore(bin): Remove `kt`

* lint

* lint

* fix(client): walkback channel timeout

* update tests

* lint

---------

Co-authored-by: clabby <ben@clab.by>
theochap pushed a commit that referenced this pull request Mar 26, 2026
theochap pushed a commit that referenced this pull request Mar 26, 2026
github-merge-queue bot pushed a commit that referenced this pull request Mar 26, 2026
* kona-client: fix trace-extension short-circuit at capped leaves

Fixes a bug where kona program could incorrectly treat a transition as
trace-extension and return early based solely on output-root equality,
even when the claimed L2 block number differed, enabling trivial wins
in dispute games.

* kona/client: Fix fmt & clippy (#456)

* kona-proof: fix TipCursor using zero output root instead of agreed prestate (#454)

This fixes an issue where the pipeline cursor was initialized with
B256::ZERO as the safe head output root. When the derivation pipeline
exhausts L1 data (EndOfSource) before deriving any new blocks, the
driver returns this zero value instead of the agreed prestate's actual
output root, causing the FPP to reject honest claims and accept zero
claims.

* fix(kona): bounded channel decompression with truncation per spec (#455)

Fixes multiple deviations in kona's channel decompression (bug #35):

Zlib: replace unbounded decompress_to_vec_zlib with
decompress_to_vec_zlib_with_limit to prevent zip-bomb OOM. On limit
hit (HasMoreOutput), keep partial output per spec instead of rejecting.

Brotli: cap output buffer growth at max_rlp_bytes_per_channel instead
of erroring when the doubled buffer exceeds the limit. Fixes false
rejection of valid channels near the size limit.

Both: on decompression errors, preserve partial output so batches
decoded before the error point are accepted (matching Go reference).

Per spec: "If the decompressed data exceeds the limit, things proceed
as though the channel contained only the first
MAX_RLP_BYTES_PER_CHANNEL decompressed bytes."

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: inphi <mlaw2501@gmail.com>
Co-authored-by: Sebastian Stammler <seb@oplabs.co>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

6 participants