Skip to content

Espresso documentation cleanup#398

Merged
philippecamacho merged 3 commits intocelo-integration-rebase-16.2from
philippe/cleanup-documentation
Apr 7, 2026
Merged

Espresso documentation cleanup#398
philippecamacho merged 3 commits intocelo-integration-rebase-16.2from
philippe/cleanup-documentation

Conversation

@philippecamacho
Copy link
Copy Markdown
Collaborator

@philippecamacho philippecamacho commented Apr 7, 2026

Closes https://app.asana.com/1/1208976916964769/project/1209976130071762/task/1213947851318197?focus=true

This PR:

  • Moves Espresso related documents from docs to espresso/docs repository
  • Removes documents that are no longer relevant.

@claude
Copy link
Copy Markdown

claude Bot commented Apr 7, 2026

Claude finished @philippecamacho's task in 42s —— View job


Review: Espresso documentation cleanup

  • Gather context and analyze changes
  • Review changed files
  • Post review feedback

Overall this is a clean documentation reorganization PR. The moves from docs/ to espresso/docs/ and the corresponding path updates in README_ESPRESSO.md look correct.

One issue found:

  • Broken link (README_ESPRESSO.md:557): The link to espresso/docs/CELO_TESTNET_MIGRATION.md will be broken — CELO_TESTNET_MIGRATION.md was deleted from docs/ but was not moved to espresso/docs/. Either remove the reference or move the file instead of deleting it. Posted as an inline comment.

Everything else (the 5 renamed/moved files, the AGENTS.md removal, and the SECURITY_ANALYSIS.md deletion) looks good.

Comment thread README_ESPRESSO.md Outdated
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reorganizes the project's documentation by moving several files to the espresso/docs/ directory and removing outdated documents, including the Celo Testnet Migration guide and a security analysis. Feedback identifies broken links in README_ESPRESSO.md caused by the deletion of the migration guide and the failure to move associated image assets along with their source files.

Comment thread README_ESPRESSO.md
Comment thread README_ESPRESSO.md Outdated
@jjeangal
Copy link
Copy Markdown
Member

jjeangal commented Apr 7, 2026

Should we also delete: pr-guidelines.md. Not sure we use it even. Actually never mind, these are upstream files.

@philippecamacho philippecamacho merged commit f402aca into celo-integration-rebase-16.2 Apr 7, 2026
46 checks passed
@philippecamacho philippecamacho deleted the philippe/cleanup-documentation branch April 7, 2026 21:29
QuentinI pushed a commit that referenced this pull request Apr 9, 2026
* Move Espresso related docs to directory espresso/docs.

* Remove obsolete documents.
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