Conversation
edonehoo
left a comment
There was a problem hiding this comment.
some moving around and tweaking, lmk what you think!
| --- | ||
| id: PatternFly CLI | ||
| title: PatternFly CLI | ||
| section: get-started |
There was a problem hiding this comment.
the get started section is more so intended for introductory/overview information. Are you set on including these docs there? Otherwise, we could move to /developer-guides or maybe at least nest it under get-started/develop (assuming this is a dev-focused tool)?
There was a problem hiding this comment.
This is for both designers and developers. At the moment it's geared towards designers more. I'm good with putting it under developer guides
| import './get-started.css'; | ||
| import { Alert } from '@patternfly/react-core'; | ||
|
|
||
| The [PatternFly CLI](https://github.com/patternfly/patternfly-cli) is a command-line tool for scaffolding projects, applying code changes, and running common project tasks. It is published as [`@patternfly/patternfly-cli` on npm](https://www.npmjs.com/package/@patternfly/patternfly-cli). |
There was a problem hiding this comment.
| The [PatternFly CLI](https://github.com/patternfly/patternfly-cli) is a command-line tool for scaffolding projects, applying code changes, and running common project tasks. It is published as [`@patternfly/patternfly-cli` on npm](https://www.npmjs.com/package/@patternfly/patternfly-cli). | |
| The [PatternFly CLI](https://github.com/patternfly/patternfly-cli) is a command-line tool published on npm as [`@patternfly/patternfly-cli`](https://www.npmjs.com/package/@patternfly/patternfly-cli). With this tool, you can scaffold projects from built-in or custom templates, automate code updates for repetitive changes, and run workflows (including init, sync with GitHub, updates, and deploys to GitHub Pages). | |
| The PatternFly CLI is developed in the open at [github.com/patternfly/patternfly-cli](https://github.com/patternfly/patternfly-cli), with release notes and tags published on [Releases page](https://github.com/patternfly/patternfly-cli/releases). To report issues or contribute to this tool, refer to the patternfly-cli repo for more instruction. |
wdyt about this description, rather than the "what you can do" list? Just to build those list points into the narrative a little more
also moved up the bit about source and releases from the end of the page
| ## What you can do | ||
|
|
||
| - **Scaffold projects** from built-in or custom templates. | ||
| - **Automate code updates** for repetitive changes. | ||
| - **Run workflows** such as init, sync with GitHub, updates, and deploys to GitHub Pages. |
There was a problem hiding this comment.
| ## What you can do | |
| - **Scaffold projects** from built-in or custom templates. | |
| - **Automate code updates** for repetitive changes. | |
| - **Run workflows** such as init, sync with GitHub, updates, and deploys to GitHub Pages. |
remove, if you're okay with folding into the intro description as I did
|
|
||
| ## Prerequisites | ||
|
|
||
| Before you install the CLI, set up: |
There was a problem hiding this comment.
| Before you install the CLI, set up: | |
| Before you install the PatternFly CLI, you must set up the following: |
| 1. **[Node.js and npm](https://nodejs.org/)** (supported versions **20–24**). | ||
| 1. **[Corepack](https://nodejs.org/api/corepack.html)** — included with Node.js; enable it with `corepack enable` after installing Node. | ||
| 1. **[GitHub CLI](https://cli.github.com/)** (`gh`) — required for commands that talk to GitHub. |
There was a problem hiding this comment.
| 1. **[Node.js and npm](https://nodejs.org/)** (supported versions **20–24**). | |
| 1. **[Corepack](https://nodejs.org/api/corepack.html)** — included with Node.js; enable it with `corepack enable` after installing Node. | |
| 1. **[GitHub CLI](https://cli.github.com/)** (`gh`) — required for commands that talk to GitHub. | |
| 1. **[Node.js and npm](https://nodejs.org/)** (versions 20–24 currently supported). | |
| 1. **[Corepack](https://nodejs.org/api/corepack.html):** After Node is installed, enabled corepack via `corepack enable` on the command line. | |
| 1. **[GitHub CLI](https://cli.github.com/):** Required for commands that refer to GitHub (`gh`). |
Is corepack enable a cli command?
| | `load` | Pull the latest changes from GitHub. | | ||
| | `deploy` | Build and deploy the app to GitHub Pages. | | ||
|
|
||
| For the most up-to-date flags and behavior, see the [PatternFly CLI README](https://github.com/patternfly/patternfly-cli/blob/main/README.md) on GitHub. |
There was a problem hiding this comment.
| For the most up-to-date flags and behavior, see the [PatternFly CLI README](https://github.com/patternfly/patternfly-cli/blob/main/README.md) on GitHub. | |
| For the most up-to-date flags and behavior guidance, refer to the [PatternFly CLI README](https://github.com/patternfly/patternfly-cli/blob/main/README.md) on GitHub. |
|
|
||
| For the most up-to-date flags and behavior, see the [PatternFly CLI README](https://github.com/patternfly/patternfly-cli/blob/main/README.md) on GitHub. | ||
|
|
||
| ## Custom templates |
There was a problem hiding this comment.
| ## Custom templates | |
| ### Custom templates |
| patternfly-cli list --template-file ./my-templates.json | ||
| ``` | ||
|
|
||
| Each entry is an object with at least `name`, `description`, and `repo` (clone URL). Optional fields include `options` (extra `git clone` arguments) and `packageManager` (`npm`, `yarn`, or `pnpm`; default is `npm`). If a custom template uses the same `name` as a built-in one, the custom definition wins. |
There was a problem hiding this comment.
| Each entry is an object with at least `name`, `description`, and `repo` (clone URL). Optional fields include `options` (extra `git clone` arguments) and `packageManager` (`npm`, `yarn`, or `pnpm`; default is `npm`). If a custom template uses the same `name` as a built-in one, the custom definition wins. | |
| Each entry is an object with at least `name`, `description`, and `repo` (clone URL). Optional fields include `options` (extra `git clone` arguments) and `packageManager` (`npm` by default, or `yarn`/`pnpm`). If a custom template uses the same `name` as a built-in template, the custom definition is used. |
|
|
||
| ## Custom templates | ||
|
|
||
| You can merge your own templates with the built-ins by passing a JSON file with `--template-file` (or `-t`): |
There was a problem hiding this comment.
| You can merge your own templates with the built-ins by passing a JSON file with `--template-file` (or `-t`): | |
| You can merge your own templates with the built-in templates by passing a JSON file with `--template-file` (or `-t`): |
| ## Source and releases | ||
|
|
||
| The CLI is developed in the open at [github.com/patternfly/patternfly-cli](https://github.com/patternfly/patternfly-cli). Report issues or contribute there; release notes and tags are published on the repository’s [Releases](https://github.com/patternfly/patternfly-cli/releases) page. |
There was a problem hiding this comment.
| ## Source and releases | |
| The CLI is developed in the open at [github.com/patternfly/patternfly-cli](https://github.com/patternfly/patternfly-cli). Report issues or contribute there; release notes and tags are published on the repository’s [Releases](https://github.com/patternfly/patternfly-cli/releases) page. |
moving up top?
There was a problem hiding this comment.
Do we even need this? I can remove it since the guidance link will take you to the readme that will have this information in it. Removing for now, if you want me to add it back let me know.
| patternfly-cli [command] | ||
| ``` | ||
|
|
||
| | Command | Description | |
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
| - **Automate code updates** for repetitive changes. | ||
| - **Run workflows** such as init, sync with GitHub, updates, and deploys to GitHub Pages. | ||
|
|
||
| ## Prerequisites |
| ## Source and releases | ||
|
|
||
| The CLI is developed in the open at [github.com/patternfly/patternfly-cli](https://github.com/patternfly/patternfly-cli). Report issues or contribute there; release notes and tags are published on the repository’s [Releases](https://github.com/patternfly/patternfly-cli/releases) page. |
There was a problem hiding this comment.
Do we even need this? I can remove it since the guidance link will take you to the readme that will have this information in it. Removing for now, if you want me to add it back let me know.
|
@edonehoo Latest updates can be found here: http://pf-cli.surge.sh/developer-guides/patternfly-cli |
edonehoo
left a comment
There was a problem hiding this comment.
some more thoughts and suggestions - lmk what you think!
| --- | ||
| id: PatternFly CLI | ||
| title: PatternFly CLI | ||
| section: developer-guides |
There was a problem hiding this comment.
| section: developer-guides | |
| section: get-started |
just saw your previous reply that this tool is actually more oriented towards designers right now. with that in mind (combined with the technicality of the doc itself), I think your initial suggestion of putting this in the get started section is okay for now actually
| section: developer-guides | ||
| --- | ||
|
|
||
| The [PatternFly CLI](https://github.com/patternfly/patternfly-cli) is a command-line tool for scaffolding projects, performing code modifications, and running project-related tasks. It aims to streamline development workflows and improve productivity. The published package is [`@patternfly/patternfly-cli` on npm](https://www.npmjs.com/package/@patternfly/patternfly-cli). |
There was a problem hiding this comment.
I did just want to mention that (as someone who has no knowledge of this tool) it's not extremely clear to me why this is a separate tool and why designers will be using this. When I read "streamline development workflows and improve productivity" it just sounds like a general dev accelerator tool, so it might be helpful to more explicitly state why we've created this new tool, why we're doing it now, and what's special about it. If I was a designer I wouldn't necessarily pick up that this tool was intended for me to use
that extra context could go up front or in the next ## Features section
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
packages/documentation-site/patternfly-docs/content/developer-guides/patternfly-cli.md
Outdated
Show resolved
Hide resolved
…guides/patternfly-cli.md Co-authored-by: Erin Donehoo <105813956+edonehoo@users.noreply.github.com>
Co-authored-by: Erin Donehoo <105813956+edonehoo@users.noreply.github.com>
Added a page discussing patternfly cli.
Resolves issue # patternfly/patternfly-cli#18