Skip to content

Conversation

@josegomezr
Copy link

Change GIT_USER_ID/GIT_REPO_ID.git for the proper GitHub repo URL.

Additionally, removed references to root permissions for pip install commands.

Change `GIT_USER_ID/GIT_REPO_ID.git` for the proper GitHub repo URL.

Additionally, removed references to root permissions for `pip install` commands.

Signed-off-by: Jose D. Gomez R. <1josegomezr@gmail.com>
@rissson
Copy link
Member

rissson commented Nov 28, 2025

This README is generated. You'll have to update the template from openapi-generator

@josegomezr
Copy link
Author

Hm, looks like I can only remove the setup.py install line, but the placeholders can't be added via config.yaml.

I could only generate a "good" readme with:

uv add "openapi-generator-cli==v7.15.0"
uv run openapi-generator-cli generate \
  --config config.yaml \
  --template-dir templates/ \
  --openapi-generator-ignore-list .openapi-generator-ignore \
  --input-spec ../authentik-source/schema.yml \
  --generator-name python \
  --git-repo-id goauthentik \
  --git-user-id client-python

partial diff on README.md (escaped backticks to keep formatting working)

@@ -30,8 +30,9 @@ Python 3.9+
 If the python package is hosted on a repository, you can install directly using:
 
 \```sh
-pip install git+https://github.com/goauthentik/client-python.git
+pip install git+https://github.com/client-python/goauthentik.git
 \```
+(you may need to run `pip` with root permission: `sudo pip install git+https://github.com/client-python/goauthentik.git`)

@rissson rissson self-requested a review December 1, 2025 15:35
@rissson rissson moved this from Todo to Needs review in authentik Core Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs review

Development

Successfully merging this pull request may close these issues.

2 participants