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

docs(cli): Added usage docs #330

Merged
merged 2 commits into from
Jul 7, 2024
Merged

docs(cli): Added usage docs #330

merged 2 commits into from
Jul 7, 2024

Conversation

rajdip-b
Copy link
Member

@rajdip-b rajdip-b commented Jul 7, 2024

PR Type

Documentation


Description

  • Added comprehensive documentation for the CLI tool, including commands like init, profile, and run.
  • Updated the table of contents to include new CLI sections and moved the architecture section under Internals.
  • Fixed image paths in the architecture documentation.
  • Added a new environment variable NEXT_PUBLIC_BACKEND_URL to the environment variables documentation.
  • Provided detailed explanations of the internal workings of the CLI, including configuration files.

Changes walkthrough 📝

Relevant files
Documentation
SUMMARY.md
Updated table of contents with CLI and Internals sections.

docs/SUMMARY.md

  • Added new sections for CLI documentation.
  • Moved the architecture section under Internals.
  • +12/-1   
    init.md
    Added documentation for the `init` CLI command.                   

    docs/cli/init.md

  • Added documentation for the init command.
  • Included usage, options, and descriptions.
  • +37/-0   
    introduction.md
    Added introduction to CLI tool and commands.                         

    docs/cli/introduction.md

  • Introduced the CLI tool and its purpose.
  • Listed available commands and global flags.
  • +31/-0   
    profile.md
    Added documentation for the `profile` CLI command.             

    docs/cli/profile.md

  • Added documentation for the profile command.
  • Included subcommands, usage, and options.
  • +127/-0 
    run.md
    Added documentation for the `run` CLI command.                     

    docs/cli/run.md

  • Added documentation for the run command.
  • Included usage and arguments.
  • +25/-0   
    environment-variables.md
    Updated environment variables documentation.                         

    docs/contributing-to-keyshade/environment-variables.md

    • Added NEXT_PUBLIC_BACKEND_URL environment variable.
    +3/-1     
    architecture-of-keyshade.md
    Fixed image paths in architecture documentation.                 

    docs/internals/architecture-of-keyshade.md

    • Fixed image paths for architecture diagrams.
    +5/-5     
    how-the-cli-works.md
    Added documentation on how the CLI works.                               

    docs/internals/how-the-cli-works.md

  • Added documentation on the internal workings of the CLI.
  • Explained configuration files and their purposes.
  • +59/-0   

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No key issues to review

    Copy link
    Contributor

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Rephrase the introduction for better clarity and readability

    To improve clarity and readability, consider rephrasing the sentence to avoid repetition
    and ensure it flows better.

    docs/cli/introduction.md [7]

    -Our CLI is one of the major component of keyshade. The CLI is the entrypoint for your application to tap into the live updates from our platform. Whatever changes you make to your secrets or variables in your project, are reflected automatically in your application. Besides that, it also helps you to interact with the keyshade API.
    +Our CLI is a major component of keyshade, serving as the entry point for your application to receive live updates from our platform. Any changes you make to your project's secrets or variables are automatically reflected in your application. Additionally, the CLI facilitates interaction with the keyshade API.
     
    • Apply this suggestion
    Suggestion importance[1-10]: 7

    Why: The suggested rephrasing improves clarity and readability by reducing repetition and enhancing the flow of the sentence.

    7
    Rephrase the sentence to avoid redundancy and improve readability

    To avoid redundancy and improve readability, consider rephrasing the sentence to eliminate
    the repeated phrase "we wanted to ensure."

    docs/internals/how-the-cli-works.md [11]

    -We wanted to ensure that the CLI is fast and efficient, so that you can get your work done quickly.
    +We aimed to make the CLI fast and efficient, enabling you to complete your tasks quickly.
     
    • Apply this suggestion
    Suggestion importance[1-10]: 6

    Why: The suggested rephrasing eliminates redundancy and makes the sentence more concise, which improves readability.

    6
    Clarify the usage of the argument in the run command documentation

    To enhance clarity, specify that the argument should be the command to run the user's
    application.

    docs/cli/run.md [19-21]

     ### `<command>`
     
    -The command to run your application. This command is run in the same shell as the `keyshade run` command.
    +The command to run your application. Replace `<command>` with the specific command you want to execute. This command is run in the same shell as the `keyshade run` command.
     
    • Apply this suggestion
    Suggestion importance[1-10]: 5

    Why: The suggestion adds a minor clarification to the <command> argument, which slightly improves the documentation but does not significantly impact understanding.

    5
    Best practice
    Adjust heading levels for subcommands to improve readability and consistency

    For better readability and consistency, consider changing the heading levels of
    subcommands under the profile command to be consistent with other sections.

    docs/cli/profile.md [21-35]

    -## `create`
    +### `create`
     
    -### Usage
    +#### Usage
     
     ```bash
     keyshade profile create [options]

    -### Options
    +#### Options

    These are the commands supported by the create command:

    -#### -n, --name <name>
    +##### -n, --name <name>

    The name of the profile. This is a required field.

    
    - [ ] **Apply this suggestion** <!-- /improve --apply_suggestion=3 -->
    
    <details><summary>Suggestion importance[1-10]: 6</summary>
    
    Why: Adjusting the heading levels improves consistency across the document, which enhances readability and document structure.
    
    
    </details></details></td><td align=center>6
    
    </td></tr></tr></tbody></table>
    
    

    @rajdip-b rajdip-b merged commit 905ff9d into develop Jul 7, 2024
    5 checks passed
    @rajdip-b rajdip-b deleted the docs/cli-usage branch July 7, 2024 13:29
    rajdip-b added a commit that referenced this pull request Jul 11, 2024
    rajdip-b pushed a commit that referenced this pull request Jul 11, 2024
    ## [2.2.0](v2.1.0...v2.2.0) (2024-07-11)
    
    ### 🚀 Features
    
    * **api-client:** Added API Client package ([#346](#346)) ([6734e1e](6734e1e))
    * **api:** Updated API key ([fbac312](fbac312))
    * **platform:** View [secure]s ([#313](#313)) ([97c4541](97c4541))
    * **web:** Add Pricing Page ([#243](#243)) ([2c7f1d6](2c7f1d6))
    
    ### 📚 Documentation
    
    * **cli:** Added docs for the CLI package ([#329](#329)) ([edad166](edad166))
    * **cli:** Added usage docs ([#330](#330)) ([b6963d5](b6963d5))
    * Update Discord link ([871b6cd](871b6cd))
    * Update README.md ([e66fcd2](e66fcd2))
    * **web:** Add documentation about our web package ([#268](#268)) ([3d848e7](3d848e7))
    
    ### 🔧 Miscellaneous Chores
    
    * **api:** Updated response types in environment service ([b8a3ddd](b8a3ddd))
    * **ci:** Added release scripts for platform and api ([02dae60](02dae60))
    * **CI:** Updated action plugin versions ([88bb317](88bb317))
    * **CI:** Updated pnpm version in CI file ([2692e88](2692e88))
    * **platform:** Fixed env parsing in platform ([d6ffafa](d6ffafa))
    * **web:** Update Terms and Conditions and Privacy Policy ([#282](#282)) ([d621dcb](d621dcb))
    
    ### 🔨 Code Refactoring
    
    * **api:** Update [secure] and variable fetching endpoints ([7d9acd0](7d9acd0))
    * **cli:** Refactored profile commands into readable blocks ([#331](#331)) ([4a8a089](4a8a089))
    * **cli:** Updated configuration commands and mechanism ([#310](#310)) ([9079b6d](9079b6d))
    rajdip-b pushed a commit that referenced this pull request Jul 11, 2024
    ## [2.2.0](v2.1.0...v2.2.0) (2024-07-11)
    
    ### 🚀 Features
    
    * **api-client:** Added API Client package ([#346](#346)) ([6734e1e](6734e1e))
    * **api:** Updated API key ([fbac312](fbac312))
    * **platform:** View [secure]s ([#313](#313)) ([97c4541](97c4541))
    * **web:** Add Pricing Page ([#243](#243)) ([2c7f1d6](2c7f1d6))
    
    ### 📚 Documentation
    
    * **cli:** Added docs for the CLI package ([#329](#329)) ([edad166](edad166))
    * **cli:** Added usage docs ([#330](#330)) ([b6963d5](b6963d5))
    * Update Discord link ([871b6cd](871b6cd))
    * Update README.md ([e66fcd2](e66fcd2))
    * **web:** Add documentation about our web package ([#268](#268)) ([3d848e7](3d848e7))
    
    ### 🔧 Miscellaneous Chores
    
    * **api:** Updated response types in environment service ([b8a3ddd](b8a3ddd))
    * **ci:** Added release scripts for platform and api ([02dae60](02dae60))
    * **CI:** Updated action plugin versions ([88bb317](88bb317))
    * **CI:** Updated pnpm version in CI file ([2692e88](2692e88))
    * **platform:** Fixed env parsing in platform ([d6ffafa](d6ffafa))
    * **web:** Update Terms and Conditions and Privacy Policy ([#282](#282)) ([d621dcb](d621dcb))
    
    ### 🔨 Code Refactoring
    
    * **api:** Update [secure] and variable fetching endpoints ([7d9acd0](7d9acd0))
    * **cli:** Refactored profile commands into readable blocks ([#331](#331)) ([4a8a089](4a8a089))
    * **cli:** Updated configuration commands and mechanism ([#310](#310)) ([9079b6d](9079b6d))
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant