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

[Sweep Rules] Update docstrings and comments in lua/hurl/wrapper.lua #6

Closed
wants to merge 5 commits into from

Conversation

sweep-ai[bot]
Copy link
Contributor

@sweep-ai sweep-ai bot commented Oct 24, 2023

PR Feedback: 👎

I created this PR to address this rule:

"All docstrings and comments should be up to date."

Description

This PR updates the docstrings and comments in the lua/hurl/wrapper.lua file to accurately reflect recent code changes. The code logic and log messages have been modified, but the corresponding documentation was not updated. This PR ensures that the docstrings and comments accurately describe the updated code logic and log messages.

Summary of Changes

  • Updated the docstring for the on_output function to include a description of the new log messages and any changes in the code logic.
  • Updated the comments within the on_output function to accurately describe the updated code logic.
  • Reviewed and updated the docstrings and comments for the request, run_current_file, and run_selection functions.
  • Reviewed and updated all other docstrings and comments in the lua/hurl/wrapper.lua file to ensure they are up to date.
  • Reviewed and updated the documentation in the doc/hurl.nvim.txt file to reflect the changes in the lua/hurl/wrapper.lua file.

Please review the changes and merge this PR to ensure that the documentation accurately reflects the updated code logic and log messages.

@sweep-ai
Copy link
Contributor Author

sweep-ai bot commented Oct 24, 2023

Rollback Files For Sweep

  • Rollback changes to lua/hurl/wrapper.lua
  • Rollback changes to lua/hurl/wrapper.lua

@sweep-ai sweep-ai bot added the sweep label Oct 24, 2023
@changeset-bot
Copy link

changeset-bot bot commented Oct 24, 2023

⚠️ No Changeset found

Latest commit: f9b8a44

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@jellydn jellydn closed this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant