Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update project config #45

Merged
merged 3 commits into from
Apr 29, 2024
Merged

Update project config #45

merged 3 commits into from
Apr 29, 2024

Conversation

nao1215
Copy link
Owner

@nao1215 nao1215 commented Apr 29, 2024

Summary by CodeRabbit

  • New Features

    • Introduced a security scanning workflow to check for sensitive information leaks in pull requests.
    • Updated code coverage reporting configurations and thresholds.
    • Added new Makefile targets for coverage reporting and changelog generation.
    • Enhanced the README with improved structure and clearer instructions for using the sqly command-line tool.
  • Documentation

    • Streamlined the Code of Conduct to emphasize respect.
    • Updated the security policy with detailed vulnerability reporting guidelines.
    • Revised CHANGELOG with latest updates and dependency changes.
  • Refactor

    • Adjusted heading levels in README for better readability and navigation.
  • Chores

    • Updated tool management configurations and dependencies.

Copy link
Contributor

coderabbitai bot commented Apr 29, 2024

Walkthrough

The project changes encompass the adoption of new tools for security and code quality, refining of the code of conduct, and enhancements to documentation and build processes. These updates aim to bolster security measures, streamline development practices, and enhance user engagement with the software.

Changes

Files Change Summary
.github/workflows/git-leaks.yml, .github/workflows/linux_test.yml Introduced git-leaks.yml for scanning sensitive info and adjusted linux_test.yml.
.golangci.yml, .octocov.yml Specified Go version, enabled linters, and set up code coverage reporting with OctoCov.
CHANGELOG.md, README.md Updated with new tools, dependencies, and improved documentation structure.
CODE_OF_CONDUCT.md, SECURITY.md Streamlined Code of Conduct and detailed security policies for the project.
Makefile, tools.go Enhanced build scripts and introduced a package for tool management.
di/wire_gen.go, domain/repository/csv.go Modified go:generate directives in the mentioned files.

🐰✨
To the code we hop and tweak,
With tools and scripts we seek,
To make our garden lush and sleek,
Bugs to banish, security peak!
Hop, hop, hurray for the geek! 🎉
🐰✨


Recent Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between 4dc702a and 861a351.
Files selected for processing (4)
  • .github/FUNDING.yml (1 hunks)
  • CHANGELOG.md (1 hunks)
  • README.md (8 hunks)
  • di/wire.go (1 hunks)
Files skipped from review due to trivial changes (2)
  • .github/FUNDING.yml
  • di/wire.go
Additional Context Used
LanguageTool (95)
CHANGELOG.md (13)

Near line 3: Possible spelling mistake found.
Context: ...re/v0.5.2...) (2024-04-29) * Introduce octocov [#44](https://github.com/nao1215/sqly/p...


Near line 4: Possible spelling mistake found.
Context: ...ps://github.com//pull/41) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 5: Possible spelling mistake found.
Context: ...ps://github.com//pull/40) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 6: Possible spelling mistake found.
Context: ...ps://github.com//pull/37) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 7: Possible spelling mistake found.
Context: ...ps://github.com//pull/36) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 8: Possible spelling mistake found.
Context: ...ps://github.com//pull/35) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 9: It appears that a hyphen is missing (if ‘auto’ is not used in the context of ‘cars’).
Context: ...https://github.com/apps/dependabot)) * (auto merged) Bump github.com/google/go-cmp from 0.5...


Near line 9: Possible spelling mistake found.
Context: ...ps://github.com//pull/34) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 10: Possible spelling mistake found.
Context: ...ps://github.com/apps/dependabot)) * Add automerged workflows [#33](https://github.com/nao1...


Near line 11: Possible spelling mistake found.
Context: ...ps://github.com//pull/32) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 12: Possible spelling mistake found.
Context: ...ps://github.com//pull/31) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 13: Possible spelling mistake found.
Context: ...ps://github.com//pull/30) ([dependabot[bot]](https://github.com/apps/dependabo...


Near line 14: Possible spelling mistake found.
Context: ...ps://github.com//pull/29) ([dependabot[bot]](https://github.com/apps/dependabo...

README.md (82)

Near line 1: Possible spelling mistake found.
Context: ...o1215/sqly/actions/workflows/build.yml) [![LinuxUnitTest](https://github.com/nao1215/sqly/action...


Near line 2: Possible spelling mistake found.
Context: .../sqly/actions/workflows/linux_test.yml) [![MacUnitTest](https://github.com/nao1215/sqly/action...


Near line 3: Possible spelling mistake found.
Context: ...15/sqly/actions/workflows/mac_test.yml) [![WindowsUnitTest](https://github.com/nao1215/sqly/action...


Near line 4: Possible spelling mistake found.
Context: ...215/sqly/actions/workflows/windows.yml) [![reviewdog](https://github.com/nao1215/sqly/action...


Near line 10: This sentence does not start with an uppercase letter.
Context: ...qly) demo sqly command imports CSV/TSV/LTSV/JSON fil...


Near line 10: Possible spelling mistake found.
Context: ...if) sqly command imports CSV/TSV/LTSV/JSON file(s) into an in-memory DB and e...


Near line 10: Possible spelling mistake found.
Context: ...DB and executes SQL against the CSV/TSV/LTSV/JSON. sqly uses [SQLite3](https://www.s...


Near line 10: This sentence does not start with an uppercase letter.
Context: ...utes SQL against the CSV/TSV/LTSV/JSON. sqly uses [SQLite3](https://www.sqlite.org/i...


Near line 10: Possible spelling mistake found.
Context: ...w.sqlite.org/index.html) as its DB. So, sql syntax is same as SQLite3. The sqly ...


Near line 12: Possible spelling mistake found.
Context: ..., sql syntax is same as SQLite3. The sqly command has sqly-shell. You can interac...


Near line 12: Possible spelling mistake found.
Context: ...ame as SQLite3. The sqly command has sqly-shell. You can interactively execute SQL with...


Near line 12: Possible spelling mistake found.
Context: ... You can interactively execute SQL with sql completion and command history. Of cour...


Near line 12: Possible spelling mistake found.
Context: ...an also execute SQL without running the sqly-shell. ## Features ✅ execute SQL against CSV...


Near line 15: Possible spelling mistake found.
Context: ...tures ✅ execute SQL against CSV / TSV / LTSV / JSON. ✅ output SQL result to CSV / ...


Near line 16: Possible spelling mistake found.
Context: ...N. ✅ output SQL result to CSV / TSV / LTSV / JSON file format. ✅ print SQL resul...


Near line 17: Possible spelling mistake found.
Context: ...SQL result in ASCII Table / CSV / TSV / LTSV / JSON file format. ✅ interactive sql...


Near line 18: Possible spelling mistake found.
Context: ...TSV / JSON file format. ✅ interactive sqly shell with input completion, emacs-keyb...


Near line 18: Possible spelling mistake found.
Context: ...ctive sqly shell with input completion, emacs-keybindings, input history. ## How to install ##...


Near line 22: The pronoun ‘you’ must be used with a non-third-person form of a verb.
Context: ... to install ### Use "go install" If you does not have the golang development environ...


Near line 22: Possible spelling mistake found.
Context: ...e "go install" If you does not have the golang development environment installed on yo...


Near line 22: Possible spelling mistake found.
Context: ...nstalled on your system, please install golang from the [golang official website](http...


Near line 22: Possible spelling mistake found.
Context: ...system, please install golang from the [golang official website](https://go.dev/doc/in...


Near line 26: Possible spelling mistake found.
Context: ...ttps://github.com/mattn/go-sqlite3) and gcc. ## How to use sqly command automatic...


Near line 29: Possible spelling mistake found.
Context: ...tn/go-sqlite3) and gcc. ## How to use sqly command automatically imports the CSV/T...


Near line 30: Possible spelling mistake found.
Context: ...mmand automatically imports the CSV/TSV/LTSV/JSON file into the DB when you pass a C...


Near line 30: Possible spelling mistake found.
Context: ...ile into the DB when you pass a CSV/TSV/LTSV/JSON file as an argument. DB table name...


Near line 30: Possible spelling mistake found.
Context: ...ile name (e.g., if you import user.csv, sqly command create the user table) ### Syn...


Near line 36: Possible spelling mistake found.
Context: ... sqly [OPTIONS] [FILE_PATH] ``` ※ The sqly option must be specified before the fil...


Near line 38: Possible spelling mistake found.
Context: ... before the file to be imported. ### --sql option: execute sql in terminal --sql o...


Near line 38: Possible spelling mistake found.
Context: ...be imported. ### --sql option: execute sql in terminal --sql option takes an SQL s...


Near line 39: Possible spelling mistake found.
Context: ...--sql option: execute sql in terminal --sql option takes an SQL statement as an opt...


Near line 39: Possible spelling mistake found.
Context: ...u pass file path(s) as arguments to the sqly command. sqly command import them. sqly...


Near line 39: This sentence does not start with an uppercase letter.
Context: ...th(s) as arguments to the sqly command. sqly command import them. sqly command autom...


Near line 39: This sentence does not start with an uppercase letter.
Context: ...sqly command. sqly command import them. sqly command automatically determines the fi...


Near line 51: Possible spelling mistake found.
Context: ...--------+ ``` ### Change output format sqly command output sql results in ASCII tab...


Near line 52: Possible spelling mistake found.
Context: ...hange output format sqly command output sql results in ASCII table format, CSV form...


Near line 52: Possible spelling mistake found.
Context: ...ts in ASCII table format, CSV format (--csv option), TSV format (--tsv option), LTS...


Near line 52: Possible spelling mistake found.
Context: ...SV format (--csv option), TSV format (--tsv option), LTSV format (--ltsv option) an...


Near line 52: Possible spelling mistake found.
Context: ...V format (--tsv option), LTSV format (--ltsv option) and JSON format (--json option)...


Near line 52: Possible spelling mistake found.
Context: ...rmat (--ltsv option) and JSON format (--json option). This means that conversion bet...


Near line 52: Possible spelling mistake found.
Context: ...on). This means that conversion between csv and json is supported. ``` $ sqly --sql...


Near line 52: Possible spelling mistake found.
Context: ...s means that conversion between csv and json is supported. ``` $ sqly --sql "SELECT ...


Near line 83: This sentence does not start with an uppercase letter.
Context: ...ker12 Mary,2,Jenkins,jenkins46 ``` ### run sqly shell If the --sql option is not s...


Near line 83: Possible spelling mistake found.
Context: ...2 Mary,2,Jenkins,jenkins46 ``` ### run sqly shell If the --sql option is not specif...


Near line 84: Possible spelling mistake found.
Context: ...kins46 ``` ### run sqly shell If the --sql option is not specified, the sqly shell...


Near line 84: Possible spelling mistake found.
Context: ... the --sql option is not specified, the sqly shell is started. When you execute sqly...


Near line 84: Possible spelling mistake found.
Context: ...sqly shell is started. When you execute sqly command, it is optional whether or not ...


Near line 84: Consider shortening this phrase to just ‘whether’, unless you mean ‘regardless of whether’.
Context: ...ou execute sqly command, it is optional whether or not to specify file(s). The sqly shell func...


Near line 84: Possible spelling mistake found.
Context: ... whether or not to specify file(s). The sqly shell functions similarly to a common S...


Near line 84: Possible spelling mistake found.
Context: ...on SQL client (e.g., sqlite3 command or mysql command). sqly shell has helper command...


Near line 84: This sentence does not start with an uppercase letter.
Context: ....g., sqlite3 command or mysql command). sqly shell has helper commands, SQL executio...


Near line 86: This sentence does not start with an uppercase letter.
Context: ... management and input complement. #### sqly helper command The command beginning wi...


Near line 87: Possible spelling mistake found.
Context: ...The command beginning with a dot is the sqly helper command; I plan to add more feat...


Near line 87: Possible spelling mistake found.
Context: ...more features in the future to make the sqly shell run more comfortably. ``` $ sqly ...


Near line 87: Did you mean “she'll”?
Context: ...features in the future to make the sqly shell run more comfortably. ``` $ sqly sqly ...


Near line 107: Possible spelling mistake found.
Context: ..../doc/img/shell-demo.png) ### Output sql result to file #### For linux user sql...


Near line 108: Possible spelling mistake found.
Context: ... ### Output sql result to file #### For linux user sqly command can save SQL executi...


Near line 108: Possible spelling mistake found.
Context: ...sql result to file #### For linux user sqly command can save SQL execution results ...


Near line 109: Possible spelling mistake found.
Context: ...o a file using shell redirection. The --csv option outputs SQL execution results in...


Near line 113: “Windows” (operating system by Microsoft) is a proper noun and needs to be capitalized.
Context: ...stdata/user.csv > test.csv #### For windows user $ sqly --sql "SELECT * FROM u...


Near line 149: Possible spelling mistake found.
Context: ... ## Features to be added - [x] import json - [x] print json format - [x] dump jso...


Near line 150: Possible spelling mistake found.
Context: ...be added - [x] import json - [x] print json format - [x] dump json file - [x] impor...


Near line 151: Possible spelling mistake found.
Context: ...son - [x] print json format - [x] dump json file - [x] import tsv - [x] ptint tsv f...


Near line 152: Possible spelling mistake found.
Context: ...ormat - [x] dump json file - [x] import tsv - [x] ptint tsv format - [x] dump tsv f...


Near line 153: Possible spelling mistake found.
Context: ...] dump json file - [x] import tsv - [x] ptint tsv format - [x] dump tsv file - [x] im...


Near line 153: Possible spelling mistake found.
Context: ... json file - [x] import tsv - [x] ptint tsv format - [x] dump tsv file - [x] import...


Near line 154: Possible spelling mistake found.
Context: ...t tsv - [x] ptint tsv format - [x] dump tsv file - [x] import ltsv - [x] print lts...


Near line 155: Possible spelling mistake found.
Context: ...format - [x] dump tsv file - [x] import ltsv - [x] print ltsv format - [x] dump lts...


Near line 156: Possible spelling mistake found.
Context: ...tsv file - [x] import ltsv - [x] print ltsv format - [x] dump ltsv file - [ ] impor...


Near line 157: Possible spelling mistake found.
Context: ...tsv - [x] print ltsv format - [x] dump ltsv file - [ ] import excel format (using [...


Near line 158: Capitalize this word if you mean the computer program “Excel”.
Context: ...ormat - [x] dump ltsv file - [ ] import excel format (using [qax-os/excelize](https:/...


Near line 158: Possible spelling mistake found.
Context: ... file - [ ] import excel format (using [qax-os/excelize](https://github.com/qax-os/exc...


Near line 158: Possible spelling mistake found.
Context: ... [ ] import excel format (using qax-os/excelize) -...


Near line 160: Possible spelling mistake found.
Context: ...ze)) - [ ] import swagger - [ ] import .gz file - [x] print markdown format - [ ] ...


Near line 161: Did you mean the formatting language “Markdown” (= proper noun)?
Context: ...agger - [ ] import .gz file - [x] print markdown format - [ ] ignore csv header option -...


Near line 162: Possible spelling mistake found.
Context: ... [x] print markdown format - [ ] ignore csv header option - [x] The file type is de...


Near line 165: Possible spelling mistake found.
Context: ... arrow, right arrow, delete char) - [x] sqly Fix problem where multiple lines of inp...


Near line 167: Possible spelling mistake found.
Context: ...ces. - [x] input completion (Tab) - [x] emacs key-bindings (Ctrl-a, Ctrl-e, Ctrl-w) -...


Near line 174: Possible spelling mistake found.
Context: ...est coverage 80% ## Unit Test Coverage Treemap treemap #...


Near line 177: Possible spelling mistake found.
Context: ...treemap ## Limitions (Not support) - DDL such as CREATE - DM...


Near line 183: Using many exclamation marks might seem excessive (in this case: 8 exclamation marks for a text that’s 5625 characters long)
Context: ...ple, GitHub Star motivates me to develop! [![Star History Chart](https://api.s...


Near line 194: Possible spelling mistake found.
Context: ...om/nao1215/sqly/issues) ## LICENSE The sqly project is licensed under the terms of ...

Additional comments not posted (1)
CHANGELOG.md (1)

15-15: Ensure the term "golden package" is correctly referenced. If it refers to a specific package, it should be properly named, possibly with a link if it's external.

Please verify the correct name and possibly add a link to the "golden package" if it refers to an external resource.


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

HOTTEST report

PASS FAIL SKIP TOTAL DURATION
181 0 0 181 35.568037337s

Reported by hottest

Copy link
Contributor

Code Metrics Report

Coverage Test Execution Time
86.1% 43s

Code coverage of files in pull request scope (87.1%)

Files Coverage
di/wire_gen.go 87.1%

Reported by octocov

@nao1215 nao1215 merged commit 77836a3 into main Apr 29, 2024
8 checks passed
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 12

@@ -0,0 +1,15 @@
## [](https://github.com/nao1215/sqly/compare/v0.5.2...) (2024-04-29)

* Introduce octocov [#44](https://github.com/nao1215/sqly/pull/44) ([nao1215](https://github.com/nao1215))
Copy link
Contributor

Choose a reason for hiding this comment

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

Correct the spelling of "octocov" to "OctoCov" to maintain consistency with official tool names.

- * Introduce octocov [#44](https://github.com/nao1215/sqly/pull/44) ([nao1215](https://github.com/nao1215))
+ * Introduce OctoCov [#44](https://github.com/nao1215/sqly/pull/44) ([nao1215](https://github.com/nao1215))

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
* Introduce octocov [#44](https://github.com/nao1215/sqly/pull/44) ([nao1215](https://github.com/nao1215))
* Introduce OctoCov [#44](https://github.com/nao1215/sqly/pull/44) ([nao1215](https://github.com/nao1215))

* Bump github.com/mattn/go-sqlite3 from 1.14.18 to 1.14.19 [#37](https://github.com/nao1215/sqly/pull/37) ([dependabot[bot]](https://github.com/apps/dependabot))
* Bump github.com/fatih/color from 1.15.0 to 1.16.0 [#36](https://github.com/nao1215/sqly/pull/36) ([dependabot[bot]](https://github.com/apps/dependabot))
* Bump github.com/mattn/go-sqlite3 from 1.14.17 to 1.14.18 [#35](https://github.com/nao1215/sqly/pull/35) ([dependabot[bot]](https://github.com/apps/dependabot))
* (auto merged) Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 [#34](https://github.com/nao1215/sqly/pull/34) ([dependabot[bot]](https://github.com/apps/dependabot))
Copy link
Contributor

Choose a reason for hiding this comment

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

Clarify the term "auto merged" by adding a hyphen to form "auto-merged" for better readability.

- * (auto merged) Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 [#34](https://github.com/nao1215/sqly/pull/34) ([dependabot[bot]](https://github.com/apps/dependabot))
+ * (auto-merged) Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 [#34](https://github.com/nao1215/sqly/pull/34) ([dependabot[bot]](https://github.com/apps/dependabot))

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
* (auto merged) Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 [#34](https://github.com/nao1215/sqly/pull/34) ([dependabot[bot]](https://github.com/apps/dependabot))
* (auto-merged) Bump github.com/google/go-cmp from 0.5.9 to 0.6.0 [#34](https://github.com/nao1215/sqly/pull/34) ([dependabot[bot]](https://github.com/apps/dependabot))


**sqly** command imports CSV/TSV/LTSV/JSON file(s) into an in-memory DB and executes SQL against the CSV/TSV/LTSV/JSON. sqly uses [SQLite3](https://www.sqlite.org/index.html) as its DB. So, sql syntax is same as SQLite3.

The sqly command has sqly-shell. You can interactively execute SQL with sql completion and command history. Of course, you can also execute SQL without running the sqly-shell.

# Features
## Features
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "## Features" heading for better readability and to adhere to Markdown best practices.

+ 
## Features

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
## Features
## Features

✅ execute SQL against CSV / TSV / LTSV / JSON.
✅ output SQL result to CSV / TSV / LTSV / JSON file format.
✅ print SQL result in ASCII Table / CSV / TSV / LTSV / JSON file format.
✅ interactive sqly shell with input completion, emacs-keybindings, input history.

# How to install
## How to install
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "## How to install" heading to improve readability and maintain consistency in Markdown formatting.

+ 
## How to install

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
## How to install
## How to install

✅ execute SQL against CSV / TSV / LTSV / JSON.
✅ output SQL result to CSV / TSV / LTSV / JSON file format.
✅ print SQL result in ASCII Table / CSV / TSV / LTSV / JSON file format.
✅ interactive sqly shell with input completion, emacs-keybindings, input history.

# How to install
## How to install
### Use "go install"
If you does not have the golang development environment installed on your system, please install golang from the [golang official website](https://go.dev/doc/install).
Copy link
Contributor

Choose a reason for hiding this comment

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

Correct the grammatical error in the sentence to improve clarity.

- If you does not have the golang development environment installed on your system, please install golang from the [golang official website](https://go.dev/doc/install).
+ If you do not have the golang development environment installed on your system, please install golang from the [golang official website](https://go.dev/doc/install).

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
If you does not have the golang development environment installed on your system, please install golang from the [golang official website](https://go.dev/doc/install).
If you do not have the golang development environment installed on your system, please install golang from the [golang official website](https://go.dev/doc/install).

@@ -50,7 +48,7 @@
+-----------+-----------+
```

## Change output format
### Change output format
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "### Change output format" heading to maintain a consistent structure throughout the document.

+ 
### Change output format

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
### Change output format

Change output format


</details>
<!-- suggestion_end -->

<!-- This is an auto-generated comment by CodeRabbit -->

@@ -82,10 +80,10 @@
Mary,2,Jenkins,jenkins46
```

## run sqly shell
### run sqly shell
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "### run sqly shell" heading to improve the readability of the document.

+ 
### run sqly shell

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
### run sqly shell

run sqly shell

If the --sql option is not specified, the sqly shell is started. When you execute sqly command, it is optional whether or not to specify file(s). The sqly shell functions similarly to a common SQL client (e.g., sqlite3 command or mysql command). sqly shell has helper commands, SQL execution history management and input complement.

### sqly helper command
#### sqly helper command
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "#### sqly helper command" subheading to separate it clearly from the preceding text.

+ 
#### sqly helper command

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
#### sqly helper command
#### sqly helper command


## Output sql result to file
### For linux user
### Output sql result to file
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a blank line above the "### Output sql result to file" heading to enhance the structure of the document.

+ 
### Output sql result to file

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
### Output sql result to file

Output sql result to file


</details>
<!-- suggestion_end -->

<!-- This is an auto-generated comment by CodeRabbit -->

sqly command can save SQL execution results to a file using shell redirection. The --csv option outputs SQL execution results in CSV format instead of table format.
```
$ sqly --sql "SELECT * FROM user" --csv testdata/user.csv > test.csv
```
### For windows user
#### For windows user
Copy link
Contributor

Choose a reason for hiding this comment

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

Capitalize "Windows" in the subheading to correct the casing and maintain professionalism in the document.

- #### For windows user
+ #### For Windows user

Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
#### For windows user
#### For Windows user

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant