Skip to content

Comments

wire-api: Remove explicit dependency on the pretty package#3651

Merged
akshaymankar merged 1 commit intodevelopfrom
sb/no-pretty
Oct 16, 2023
Merged

wire-api: Remove explicit dependency on the pretty package#3651
akshaymankar merged 1 commit intodevelopfrom
sb/no-pretty

Conversation

@stefanwire
Copy link
Contributor

@stefanwire stefanwire commented Oct 13, 2023

proto-lens already provides wrappers around the usage, so we should stick to those.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d No changelog.
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Oct 13, 2023
@akshaymankar akshaymankar changed the title remove explicit pretty (printer) dependency wire-api: Remove explicit dependency on the pretty package Oct 16, 2023
@akshaymankar akshaymankar marked this pull request as ready for review October 16, 2023 10:08
@akshaymankar akshaymankar merged commit 3f01001 into develop Oct 16, 2023
@akshaymankar akshaymankar deleted the sb/no-pretty branch October 16, 2023 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants