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: service account updates #871

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

krushanbauva
Copy link
Contributor

Description 📝

Quick Links 🚀

Assertion Tests 🤖

@krushanbauva krushanbauva self-assigned this Feb 4, 2025
@hasura-bot
Copy link
Contributor

hasura-bot commented Feb 4, 2025

DX: Assertion Testing

✅ Diff

The change in the command line flag from '--pat' to '--access-token' is correctly updated, ensuring the correct terminology is used in the context of service accounts. This modification clarifies to the user the type of token to be provided when authenticating via the CLI.

✅ Integrated

The updated terminology in the PR integrates well with the entire documentation and reinforces the usage of service account tokens over Personal Access Tokens (PAT) for CI/CD workflows. This change enhances the overall usability by providing consistency and helps prevent any potential confusion for users interacting with service accounts through the CLI.

@hasura-bot
Copy link
Contributor

@krushanbauva Thanks for your PR! I've assigned @seanparkross to review it.

Copy link

cloudflare-workers-and-pages bot commented Feb 4, 2025

Deploying ddn-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: ba81550
Status: ✅  Deploy successful!
Preview URL: https://76b71123.v3-docs-eny.pages.dev
Branch Preview URL: https://krushan-service-account-upda.v3-docs-eny.pages.dev

View logs

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.

2 participants