Skip to content

Conversation

@wojtekn
Copy link
Contributor

@wojtekn wojtekn commented Dec 17, 2024

I propose to add support for JSON format to make parsing and validation easier.

@wojtekn wojtekn requested a review from a team December 17, 2024 16:48
Copy link
Contributor

@fredrikekelund fredrikekelund left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Member

@sejas sejas left a comment

Choose a reason for hiding this comment

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

It works as expected and when running wp sqlite tables --format=json it produced this output:

[
  'wp_users',
  'wp_usermeta',
  'wp_termmeta',
  'wp_terms',
  'wp_term_taxonomy',
  'wp_term_relationships',
  'wp_commentmeta',
  'wp_comments',
  'wp_links',
  'wp_options',
  'wp_postmeta',
  'wp_posts'
]

@wojtekn wojtekn merged commit b3b9839 into main Dec 18, 2024
28 checks passed
@sejas sejas deleted the add/support-for-json-format branch December 18, 2024 17:11
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.

4 participants