Refactor Code for Consistency and Style Guide Adherence

sweep-ai[bot] commented 10 months ago

PR Feedback: 👎

I created this PR to address this rule:

"Code should be properly formatted and follow a consistent style guide."

Description

This PR addresses inconsistencies in the codebase and violations of the style guide. The changes include moving functions to appropriate files, updating import statements, and modifying function signatures and command usage.

Summary of Changes

Moved is_git_repo, get_git_root, and split_path functions from main.lua to git_utils.lua.
Moved find_http_verb and find_http_verb_positions_in_buffer functions from main.lua to http_utils.lua.
Updated import statements in main.lua to reflect the new locations of the moved functions.
Updated the function signatures of run_lines and run_at_lines in main.lua to match their new signatures in http_utils.lua.
Updated the command registration and usage for the HurlVerbose command in main.lua.

These changes improve the organization of the code and ensure adherence to the style guide.

sweep-ai[bot] commented 10 months ago

Rollback Files For Sweep

[ ] Rollback changes to lua/hurl/http_utils.lua
[ ] Rollback changes to lua/hurl/http_utils.lua
[ ] Rollback changes to lua/hurl/main.lua
[ ] Rollback changes to lua/hurl/main.lua

sweep-ai[bot] commented 10 months ago

Sandbox Executions

[X] Ran sandbox for lua/hurl/git_utils.lua. ✗

Run `lua/hurl/git_utils.lua` through the sandbox.
[X] Ran sandbox for lua/hurl/http_utils.lua. ✓

Sandbox logs for https://github.com/jellydn/hurl.nvim/commit/a8c2045e26881fe87f079dfaacf789c8fedff3aa

trunk fmt lua/hurl/http_utils.lua || exit 0 1/2 ✓
 ✔ Formatted lua/hurl/http_utils.lua
Re-checking autofixed files...

Checked 1 file
✔ No issues
trunk check --fix --print-failures lua/hurl/http_utils.lua 2/2 ✓
Checked 1 file
✔ No issues
Run trunk upgrade to upgrade 4 linters

[X] Ran sandbox for lua/hurl/main.lua. ✓

Sandbox logs for

trunk fmt lua/hurl/main.lua || exit 0 1/2 ✓
Checked 1 file
✔ No issues
trunk check --fix --print-failures lua/hurl/main.lua 2/2 ✓
Checked 1 file
✔ No issues

changeset-bot[bot] commented 10 months ago

⚠️ No Changeset found

Latest commit: 88e78c1fbcaa4afef0df30b3a6aa2d45e595b886

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 / hurl.nvim