-
Notifications
You must be signed in to change notification settings - Fork 58
docs: add system requirements section to README #3350
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
base: main
Are you sure you want to change the base?
Changes from all commits
97319e3
4660705
c4602e3
34c03f2
e1787a7
4ed8095
bd73ac7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,6 +11,19 @@ such as: | |
| Consult the [documentation][docs] for available sub-commands, descriptions and | ||
| examples of use. | ||
|
|
||
| ## Development Requirements | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [low] naming-convention Section header Development Requirements uses a noun phrase, while some other sections use gerund forms (Building, Testing, Linting). However, existing sections Demo and Troubleshooting are also noun phrases, so the naming style is already inconsistent in the current README. Suggested fix: Consider renaming to Prerequisites for brevity, but this is purely stylistic and the existing README already mixes naming conventions. |
||
|
|
||
| Before building or testing `ec`, install the following tools: | ||
|
|
||
| - Go 1.25.8+ (matching `go.mod`) | ||
|
ded-furby marked this conversation as resolved.
|
||
| - Make | ||
| - Podman or Docker (for acceptance tests and container-based workflows) | ||
| - Node.js 18+ (for `tekton-lint`) | ||
|
|
||
|
ded-furby marked this conversation as resolved.
|
||
| On Fedora/RHEL systems, acceptance tests often require Podman to run as a user service. | ||
| Use the troubleshooting section below for the required Podman service commands, and | ||
| then run the appropriate Building or Testing commands. | ||
|
|
||
| ## Building | ||
|
|
||
| Run `make build` from the root directory and use the `dist/ec` executable, or | ||
|
|
@@ -78,7 +91,7 @@ It is advised to do a reboot afterwards. | |
| To start Podman for the user: | ||
|
|
||
| ``` bash | ||
| $ systemctl enable --user --now podman.socket podman.service | ||
| $ systemctl enable --user --now podman.socket | ||
| ``` | ||
|
|
||
| #### **2.2. Get localhost:37837: connection reset by peer** | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.