FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fixup(readme): Align references to git repo url by josegomezr · Pull Request #8 · goauthentik/client-python · GitHub

fixup(readme): Align references to git repo url - #8

Open
josegomezr wants to merge 2 commits into
goauthentik:mainfrom
josegomezr:patch-1
Open

fixup(readme): Align references to git repo url#8
josegomezr wants to merge 2 commits into
goauthentik:mainfrom
josegomezr:patch-1

Conversation

Copy link
Copy Markdown

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 commented Nov 28, 2025

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
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 self-requested a review December 1, 2025 15:35
rissson moved this from Todo to Needs review in authentik Core Dec 1, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL