Revise EIP-2535

[Pandapip1]

CC: @mudgen

[eth-bot]

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

---

(fail) eip-2535.md

classification
updateEIP

• eip-2535.md requires approval from one of (@mudgen)

[MicahZoltu]

We should get this to align with latest template before merging. Remove Simple Summary, add description header field, correct sections to match EIP sections, remove external links, etc.

At the moment, this doesn't look much like an EIP at all, and I actually recommend moving it back to Review as it will need a lot of work to bring it in line with our repository standards.

[MicahZoltu]

I'm not sure why I approved this. 😬 This still doesn't follow the template at all. Lots of sections that aren't part of the templated sections for example. Also the comment above about order of metadata fields needs to be fixed.

[lightclient]

Looks pretty good overall, just a few nits.

[eip-automerger]

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

• Trying to change EIP 2535 state from Last Call to Review
• EIP 2535 requires approval from one of (@mudgen)

[Pandapip1]

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

Trying to change EIP 2535 state from Last Call to Review
EIP 2535 requires approval from one of (@mudgen)

Is @eip-automerger the new @eth-bot?

[Pandapip1]

@mudgen Please note that I have removed a lot of content from this. This is primarily so that this PR can be merged, but feel free to re-add things like the spec for Diamond Storage & App Storage.

I might also consider adding myself as an author. LMK if that's okay!

Finally, I also feel like the spec could be tighter overall. I would recommend bundling up many of the conventions and making them extensions. Notably, I would also like if the names were more strict, to aid with interfacing. This is outside of the scope of this PR, though.

[mudgen]

@Pandapip1 I appreciate your help but I don't want to make a lot of the changes in this pull request. I do not want to rewrite or remove what is already written. But I do want changed some of the header names and other smaller changes to make it more compliant with the EIP template.

[MicahZoltu]

Is this the new @eth-bot?

Depends on what you mean by "new". 😀 It is the auto-merge bot letting you know what needs to change in order to merge this PR.

[Pandapip1]

I'll let @mudgen do this.