8000 Initial CLI docs by gemanor · Pull Request #566 · permitio/docs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Initial CLI docs #566

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Initial CLI docs #566

wants to merge 4 commits into from

Conversation

gemanor
Copy link
Collaborator
@gemanor gemanor commented Jun 18, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings June 18, 2025 11:31
Copy link
netlify bot commented Jun 18, 2025

Deploy Preview for permitio-docs ready!

Name Link
🔨 Latest commit 04a4e18
🔍 Latest deploy log https://app.netlify.com/projects/permitio-docs/deploys/6852c3d512254b00082c4bc6
😎 Deploy Preview https://deploy-preview-566--permitio-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds the initial Permit CLI quickstart documentation and updates sidebar ordering and metadata across overview guides.

  • Introduces a new permit-cli-quickstart doc and links it in the sidebar
  • Bumps sidebar_position, titles, descriptions, and progress indicators for existing overview pages
  • Adds a reference entry for the CLI under the SDK sidebar

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sidebars.js Insert CLI quickstart into overview and CLI reference under SDK
docs/overview/sync-your-first-user-with-sdk.mdx Incremented sidebar position
docs/overview/sync-applications-data.mdx Updated sidebar position, title, description, and WhatsNext progress
docs/overview/setup-attribute-based-access-control.mdx Updated sidebar position, title, description, and WhatsNext progress
docs/overview/permit-cli-quickstart.mdx Added new quickstart guide for Permit CLI
docs/overview/perform-policy-check-with-cloud-pdp.mdx Incremented sidebar position and added CLI link in WhatsNext
docs/overview/local-authorization-microservice.mdx Updated sidebar position, title, description, and WhatsNext progress
docs/overview/create-a-rebac-policy.mdx Updated sidebar position, added description, and WhatsNext progress
docs/overview/advanced-authorization-queries.mdx Updated sidebar position, added description, and WhatsNext progress
docs/overview/access-requests-and-approvals.mdx Updated sidebar position, title, and description
Comments suppressed due to low confidence (1)

docs/overview/permit-cli-quickstart.mdx:230

  • This checklist item mentions audit replay and end-to-end tests, but the walkthrough never covers those steps; either remove it or add the corresponding instructions.
- ✅ Tested your policies with audit replay and end-to-end tests

@@ -141,7 +142,7 @@ resources—if the conditions (like their role, location, and task type) align p
policy. This adaptability makes ABAC a popular choice for modern, cloud-based systems
:::

<WhatsNext progress={60} customText="You are now a master of ABAC!">
<WhatsNext progress={70} customText="You are now a master of ABAC!">
Copy link
Preview
Copilot AI Jun 18, 2025

Choose a reason for hiding this comment

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

[nitpick] Progress percentages are updated manually across multiple docs and can easily drift; consider deriving this value dynamically or centralizing it to reduce maintenance overhead.

Suggested change
<WhatsNext progress={70} customText="You are now a master of ABAC!">
const totalSteps = 5; // Total number of steps in the guide
const completedSteps = 5; // Number of steps completed so far
const progress = Math.round((completedSteps / totalSteps) * 100);
<WhatsNext progress={progress} customText="You are now a master of ABAC!">

Copilot uses AI. Check for mistakes.

Copy link
argos-ci bot commented Jun 18, 2025

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ⚠️ Changes detected (Review) 21 changed, 2 added Jun 18, 2025, 1:58 PM

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.

1 participant
0