Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add optimism sepolia testnet [APE-1488] #105

Merged
merged 13 commits into from
Dec 13, 2023
Merged

Conversation

fubuloubu
Copy link
Member

@fubuloubu fubuloubu commented Nov 1, 2023

What I did

requires: ApeWorX/ape-optimism#19

How I did it

How to verify it

Checklist

  • Passes all linting checks (pre-commit and CI jobs)
  • New test cases have been added and are passing
  • Documentation has been updated
  • PR title follows Conventional Commit standard (will be automatically included in the changelog)

@vany365 vany365 changed the title feat: add optimism sepolia testnet feat: add optimism sepolia testnet [APE-1488] Nov 1, 2023
Copy link

linear bot commented Nov 1, 2023

APE-1488 "feat: add optimism sepolia testnet" (ApeWorX/ape-etherscan #105)

What I did

fixes: #

How I did it

How to verify it

Checklist

  • Passes all linting checks (pre-commit and CI jobs)
  • New test cases have been added and are passing
  • Documentation has been updated
  • PR title follows Conventional Commit standard (will be automatically included in the changelog)

ApeWorX/ape-etherscan #105 by fubuloubu on GitHub

via LinearSync

Copy link

github-actions bot commented Dec 2, 2023

This pull request is considered stale because it has been open 30 days with no activity. Remove stale label, add a comment, or make a new commit, otherwise this PR will be closed in 5 days.

@github-actions github-actions bot added the stale label Dec 2, 2023
@fubuloubu fubuloubu removed the stale label Dec 4, 2023
"mainnet",
"goerli",
"sepolia",
Copy link
Member

@antazoey antazoey Dec 12, 2023

Choose a reason for hiding this comment

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

only addition besides ethereum's here, rest of diff is sorting
note: eth also just has sep added

antazoey
antazoey previously approved these changes Dec 12, 2023
@antazoey antazoey enabled auto-merge (squash) December 12, 2023 22:18
@fubuloubu
Copy link
Member Author

What's going on with that new tokens file?

@antazoey
Copy link
Member

What's going on with that new tokens file?

no idea - deleted though

@antazoey
Copy link
Member

tests were broken in a few ways - also was an issue with version types.
all should be golden now.

@antazoey antazoey enabled auto-merge (squash) December 13, 2023 00:51
@antazoey antazoey merged commit bcf9a65 into main Dec 13, 2023
24 checks passed
@antazoey antazoey deleted the feat/add-optimism-sepolia branch December 13, 2023 00:58
antazoey added a commit to NotPeopling2day/ape-etherscan that referenced this pull request Dec 13, 2023
Co-authored-by: Juliya Smith <jules@apeworx.io>
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