Skip to content

Error_json: move to stdlib #17236

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

Open
wants to merge 2 commits into
base: compatible
Choose a base branch
from

Conversation

dannywillems
Copy link
Member

@dannywillems dannywillems commented May 21, 2025

Simplifying the number of OPAM libraries we do expose. It should be minimal, in particular if we want one day to publish on opam-repository.

The stdlib should contain code that do not depend on other part of the codebase, but still plays a significant role in the Mina distribution.

@dannywillems dannywillems requested review from a team as code owners May 21, 2025 03:52
@dannywillems
Copy link
Member Author

!ci-build-me

@dannywillems
Copy link
Member Author

!ci-bypass-changelog

@georgeee georgeee force-pushed the dw/move-error-json-into-mina-stdlib branch from a566877 to 59833ed Compare May 21, 2025 10:29
@georgeee
Copy link
Member

Forcepushed with fixed formatting (ran make reformat)

@georgeee
Copy link
Member

!ci-build-me

@dannywillems dannywillems force-pushed the dw/move-error-json-into-mina-stdlib branch from 59833ed to be7bc01 Compare May 23, 2025 07:57
@dannywillems
Copy link
Member Author

!ci-build-me

@dannywillems
Copy link
Member Author

Force pushed after rebase, fixing the conflicts.

@dannywillems
Copy link
Member Author

Fixed dune file in testing/integration_test_local_engine

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