Skip to content

Conversation

@tk120404
Copy link

@tk120404 tk120404 commented Jan 5, 2026

Removed an extra closing brace in the cmdHooks function that was being appended to the JSON response buffer.

Please do not open a pull request without first filing an issue and/or discussing the feature directly with the project maintainer.

Please ensure you adhere to every item in this list

  • This PR was pre-approved by the project maintainer
  • I have self-reviewed the code
  • I have added all necessary tests

Describe your changes

Please provide detailed description of the changes.

Issue number and link

Pull request require a prior issue with discussion.
Include the issue number of link here.

Removed an extra closing brace in the cmdHooks function that was being appended to the JSON response buffer.
Copy link
Contributor

@iwpnd iwpnd left a comment

Choose a reason for hiding this comment

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

The first closes the object in line 405, the second closes the meta object. Please do not create PRs without first creating an issue first.

@tidwall
Copy link
Owner

tidwall commented Jan 6, 2026

@iwpnd is right. With this change make test will fail with an 'invalid json' error.

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.

3 participants