Skip to content

[docs] Update documentation for features from 2026-04-23#862

Closed
danielmeppiel wants to merge 1 commit intomainfrom
docs/update-changelog-pr804-2026-04-23-0eaf51dc55c567f0
Closed

[docs] Update documentation for features from 2026-04-23#862
danielmeppiel wants to merge 1 commit intomainfrom
docs/update-changelog-pr804-2026-04-23-0eaf51dc55c567f0

Conversation

@danielmeppiel
Copy link
Copy Markdown
Collaborator

Documentation Updates - 2026-04-23

This PR updates the documentation based on features merged in the last 24 hours.

Features Documented

Changes Made

  • Added missing ### Fixed entry in CHANGELOG.md under [Unreleased] for the generic host port fix

Merged PRs Referenced

Notes

PR #804's commit message explicitly stated it added a CHANGELOG entry under [Unreleased], but the entry was absent from the merged state. This PR adds the missing entry:

Generic host clone and ls-remote error messages now include the custom port (host:port) so users on Bitbucket Datacenter or other self-hosted servers running on non-default ports see the exact address they need to verify against their git credential helper.

No other documentation files required changes — the existing docs/src/content/docs/guides/private-packages.md already documents custom port syntax for self-hosted servers, and this fix is an error-message quality improvement rather than a feature or API change.

Generated by Daily Documentation Updater ·

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/daily-doc-updater.md@b87234850bf9664d198f28a02df0f937d0447295
  • expires on Apr 25, 2026, 3:57 AM UTC

The [Unreleased] section was missing the Fixed entry documenting
that clone and ls-remote error messages now surface host:port for
self-hosted servers on non-default ports (e.g. Bitbucket Datacenter).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@danielmeppiel danielmeppiel added automation Deprecated: use type/automation. Kept for issue history; will be removed in milestone 0.10.0. documentation Deprecated: use type/docs. Kept for issue history; will be removed in milestone 0.10.0. labels Apr 23, 2026
Copilot AI review requested due to automatic review settings April 23, 2026 03:57
@danielmeppiel danielmeppiel enabled auto-merge April 23, 2026 03:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the missing [Unreleased] changelog entry documenting the previously-merged fix that now surfaces host:port in generic-host clone / ls-remote error messages (PR #804, closes #798).

Changes:

  • Add an [Unreleased] ### Fixed section entry describing the generic-host custom port error-message improvement.
Show a summary per file
File Description
CHANGELOG.md Adds the missing Unreleased “Fixed” entry for the generic-host host:port error-message fix (PR #804).

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown

APM Review Panel Verdict

Disposition: APPROVE

PR: #862 -- [docs] Update documentation for features from 2026-04-23
Change: +4 lines in CHANGELOG.md only -- backfills missing ### Fixed entry for PR #804 (generic host host:port surfacing in clone/ls-remote error messages).


Per-persona findings

  • Python Architect: No code changes. The added entry slots correctly under [Unreleased] after the existing ### Added block. Observed a pre-existing issue unrelated to this PR: [0.9.1] contains two separate ### Fixed headers (one for fix(workflows): allow fork PRs in pr-review-panel #826/fix(panel-workflow): only fire on label, add dispatch path for fork PRs #836/fix(panel-workflow): use pull_request_target so labelled fork PRs run #837, one for fix(install): lowercase host in fallback-port-warned dedup key #815). Should be consolidated in a follow-up, but this PR must not be blocked for it.

  • CLI Logging Expert: No CLI output or logger changes. Not applicable as a code reviewer. The CHANGELOG entry prose is clear and user-facing ("the exact address they need to verify against their git credential helper") -- no jargon debt introduced.

  • DevX UX Expert: Entry reads well for the target persona (developer on Bitbucket Datacenter / self-hosted git with non-standard port). It names the symptom, the fix, and the affected tooling without over-explaining. Good changelog ergonomics.

  • Supply Chain Security Expert: CHANGELOG.md is inert documentation -- no dependency resolution, lockfile, path construction, or token-handling surface touched. No concerns.

  • Auth Expert: Not applicable. The entry references "git credential helper" as an end-user concept, not a code-level change. No auth logic is modified.

  • OSS Growth Hacker: The explicit "Bitbucket Datacenter" callout is a minor adoption signal -- it tells enterprise self-hosted users APM is aware of their stack. No growth-strategy.md update required for a CHANGELOG backfill, but the fix itself (fix(install): surface custom port in generic host clone/ls-remote error #804) is worth highlighting in the next release notes pass as evidence of enterprise-grade error ergonomics.


CEO arbitration

This is a clean, automated documentation maintenance PR. The sole change is a four-line CHANGELOG backfill for a bug fix that was confirmed merged (#804) but whose CHANGELOG entry was absent from the resulting commit. The entry format is correct (Keep a Changelog convention, PR reference, closes-issue link). All seven panel members have no blocking concerns. The pre-existing duplicate ### Fixed in [0.9.1] is a real structural defect, but it predates this PR and must be addressed in a dedicated cleanup, not as a gate on this merge. Auto-merge is the right call.


Required actions before merge

None. Auto-merge is already enabled and is appropriate for this change.


Optional follow-ups

  • Open a follow-up PR to consolidate the two ### Fixed headers under [0.9.1] into one (currently lines 31-37 in CHANGELOG.md). This is a pre-existing formatting defect introduced before [docs] Update documentation for features from 2026-04-23 #862 and is cosmetic but worth fixing for changelog tooling correctness.

Generated by PR Review Panel for issue #862 · ● 456.7K ·

danielmeppiel added a commit that referenced this pull request Apr 23, 2026
…ict (#867)

Merge Gate now waits for ALL 5 PR-time required checks (Build & Test
(Linux) from ci.yml, plus Build/Smoke/Integration/Release-Validate
(Linux) stubs from ci-integration-pr-stub.yml) and emits one verdict.
This is the Tide / bors pattern: one gate to rule them all.

Once this PR merges, branch protection can be flipped to require ONLY
'Merge Gate / gate'. The other 5 checks remain informational. This
decouples the protection ruleset from CI workflow topology -- adding
or renaming an underlying check no longer requires a ruleset edit.

Script changes:
- EXPECTED_CHECK (single string) -> EXPECTED_CHECKS (comma-separated).
- Per-check state in parallel indexed arrays (works on bash 3.2+, no
  associative arrays required).
- Fail-fast on the first failing check (exit 1 with annotation).
- Timeout categorizes missing (never appeared) vs stuck (appeared but
  did not complete) and emits distinct error messages.
- Same exit-code semantics as before, applied to the aggregate.

Workflow changes:
- Pass the 5 expected check names via EXPECTED_CHECKS env.
- 'Merge Gate / gate' is excluded from the wait list by construction
  (it would deadlock waiting for itself).

Tested live against PR #862 head SHA (all 5 OK -> exit 0) and against
SHA 0000... (all 5 missing -> exit 2 with clear error annotation).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

This pull request was automatically closed because it expired on 2026-04-25T03:57:11.612Z.

Closed by Workflow

@github-actions github-actions Bot closed this Apr 25, 2026
auto-merge was automatically disabled April 25, 2026 06:59

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation Deprecated: use type/automation. Kept for issue history; will be removed in milestone 0.10.0. documentation Deprecated: use type/docs. Kept for issue history; will be removed in milestone 0.10.0.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] is_generic clone/ls-remote error path renders bare host, losing custom port for Bitbucket DC users

2 participants