docs: consolidate code hosting setup instructions#19202
docs: consolidate code hosting setup instructions#19202nijel wants to merge 1 commit intoWeblateOrg:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Consolidates provider-specific code hosting integration guidance (repo access, notifications, and push/PR workflows) into a single administrator-facing documentation page, and updates existing docs to link to it (Fixes #16257).
Changes:
- Added a new provider-oriented admin doc page:
/admin/code-hosting. - Replaced scattered provider-specific setup sections in existing docs with concise pointers to the new page.
- Added the new page to the docs index and noted it in the changelog.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/admin/code-hosting.rst | New consolidated provider checklist for GitHub/GitLab/Gitea/Bitbucket/Azure/Pagure/etc. |
| docs/vcs.rst | Replaces provider-specific sections with links to the consolidated page. |
| docs/admin/continuous.rst | Replaces webhook/app setup blocks with a pointer to the consolidated page. |
| docs/admin/config.rst | Adds a cross-link from credential settings to the consolidated page. |
| docs/index.rst | Adds the new page to the Administrator docs toctree. |
| docs/changes.rst | Mentions the new documentation page in the 5.17.1 notes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b87d40f7e7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Move scattered pieces to a single location. Fixes WeblateOrg#16257
💡 Codex ReviewLine 140 in beb5c9f This edits the already released Weblate 5.17 changelog section, which conflicts with the repository guidance in ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
I think it would be also beneficial to include how to push changes from Weblate, especially the table there (now available at https://docs.weblate.org/en/latest/admin/continuous.html#pushing-changes-from-weblate ). I am not sure where to place it, though. Either after the Setup overview or at the very bottom? Or should it be referenced in each VCS section? What does @orangesunny think? |
| :guilabel:`Webhook` in GitHub. It does not by itself grant Hosted Weblate write | ||
| access to the repository, though. To push changes back, you still need to add |
There was a problem hiding this comment.
| :guilabel:`Webhook` in GitHub. It does not by itself grant Hosted Weblate write | |
| access to the repository, though. To push changes back, you still need to add | |
| :guilabel:`Webhook` in GitHub. However, it does not by itself grant Hosted Weblate write | |
| access to the repository. To push changes back, you still need to add |
Move scattered pieces to a single location.
Fixes #16257