Sam Wilson

Require that `discussions-to` points to Ethereum Magicians

15

As opposed to a GitHub issue or back to the PR itself.

Set `last-call-deadline` in the preamble before merging relevant pull requests

13

_Originally from #55_ Immediately before merging (IMO, ideally in the merge commit itself) set `last-call-deadline` in the preamble to 14 days after the merge date.

Set created date in the preamble immediately before merging new EIP pull requests

9

_Originally from #55_ Immediately before merging (IMO, ideally in the merge commit itself) set the created date in the preamble.

Set EIP number immediately before merging new EIP pull requests

8

_Originally from #55_ Immediately before merging (IMO, ideally in the merge commit itself) set the EIP number in the preamble and rename the markdown file. How that number is determined...

Pull Request labelled with `updateEIP` instead of `newEIPFile`

1

See https://github.com/ethereum/EIPs/pull/5313. Reported by @abcoathup on discord.

`more than one target found for cross-reference` warning appeared after upgrading to `sphinx==5.2.3` and `sphinx-autoapi>=2.0.0`

1

Hey! Our build is popping up a new warning after upgrading to the latest sphinx and autoapi. ``` /home/sam/Code/specs/autoapi_mre/doc/autoapi/file3/index.rst:33:more than one target found for cross-reference 'Transaction': file1.Transaction, file2.Transaction ``` Here's...

Warn/fail if a proposal is created in `Review` (or later) status

4

Downstream tooling requires each proposal be merged as a Draft first, then move to Review. If the proposal appears first in Review, it messes up statistics. _Originally reported by @poojaranjan...

Editors shouldn't count as their own approval

5

For example, @lightclient is both an author and an editor: https://github.com/ethereum/EIPs/pull/6642

For example, from imToken: 

To support more wallets (especially mobile) we should support WalletConnect-based wallets. One option is [Web3Modal](https://github.com/WalletConnect/web3modal).