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

added show to registry #890

Closed
wants to merge 2 commits into from
Closed

added show to registry #890

wants to merge 2 commits into from

Conversation

esohel30
Copy link
Contributor

#869

This pull request adds pp and show functions to the Registry module to enable pretty-printing of the map contents for easier debugging and logging.

Changes:

Added pp Function:

The pp function formats the map's key-value pairs for pretty-printing.
Utilizes Format.fprintf for formatting and printing the map contents in a readable format.

Added show Function:

The show function converts the map to a string representation using the pp function.

Uses Format.asprintf to generate the string representation of the map.

@esohel30
Copy link
Contributor Author

Hey maxime I am kind of confused about pretty-printing a registry. How do I go about doing it within a test. I understand that I don't have to make it a test and that I just need to demonstrate that it works by showing some logs/stderr output but I am still confused on how to go about doing that.

@arthaud arthaud closed this Jul 18, 2024
@arthaud
Copy link
Contributor

arthaud commented Jul 18, 2024

Done in #877

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants