Skip to content

copilot instructions: reword no prerelease docs#7101

Merged
jamadeo merged 2 commits intomainfrom
no-prerelease-docs-take-trois
Feb 26, 2026
Merged

copilot instructions: reword no prerelease docs#7101
jamadeo merged 2 commits intomainfrom
no-prerelease-docs-take-trois

Conversation

@dianed-square
Copy link
Copy Markdown
Contributor

Summary

This PR revises the "No Prerelease Docs" rule to more clearly explain the timing gap

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)
    • Copilot PR review instructions

AI Assistance

  • This PR was created or reviewed with AI assistance

Copilot AI review requested due to automatic review settings February 9, 2026 23:37
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

Updates the repository’s Copilot review instructions to clarify guidance around documentation changes being merged ahead of code releases.

Changes:

  • Renames the “No Prerelease Docs” section to “No Doc Updates with Code Changes”.
  • Rewords the rule to emphasize the merge-vs-release timing gap and suggests using unlisted: true or hiding docs.

Comment on lines +37 to +38
### No Doc Updates with Code Changes
- PRs with code changes shouldn't update `/documentation` - docs deploy on merge, code on release. Use `unlisted: true` or remove/hide docs.
Copy link

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

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

This change appears to broaden the rule from "code changes to features/functionality" to effectively "any code changes", which is a behavior change (not just a rewording) and may incorrectly block doc fixes that should ship with already-released behavior; consider keeping the original condition or explicitly scoping it to unreleased/behind-release features.

Copilot uses AI. Check for mistakes.
@jamadeo jamadeo added this pull request to the merge queue Feb 26, 2026
Merged via the queue into main with commit 835a1af Feb 26, 2026
24 checks passed
@jamadeo jamadeo deleted the no-prerelease-docs-take-trois branch February 26, 2026 16:10
tlongwell-block added a commit that referenced this pull request Feb 27, 2026
…patible

* origin/main: (70 commits)
  feat: allow goose askai bot to search goose codebase (#7508)
  Revert "Reapply "fix: prevent crashes in long-running Electron sessions""
  Reapply "fix: prevent crashes in long-running Electron sessions"
  Revert "fix: prevent crashes in long-running Electron sessions"
  fix: replace unwrap() with graceful error in scheduler execute_job (#7436)
  fix: Dictation API error message shows incorrect limit (#7423)
  fix(acp): Use ACP schema types for session/list (#7409)
  fix(desktop): make bundle and updater asset naming configurable (#7337)
  fix(openai): preserve order in Responses API history (#7500)
  Use the correct Goose emoji 🪿 instead of Swan in README.md (#7485)
  feat(ui): implement fullscreen and pip display modes for MCP Apps (#7312)
  fix: prevent crashes in long-running Electron sessions
  Disable tool pair summarization (#7481)
  fix: New Recipe Warning does not close on cancel (#7524)
  The client is not the source of truth (#7438)
  feat: support Anthropic adaptive thinking (#7356)
  copilot instructions: reword no prerelease docs (#7101)
  fix(acp): don't fail session creation when model listing is unavailable (#7484)
  feat: simplify developer extension (#7466)
  feat: add goose-powered release notes generator workflow (#7503)
  ...

# Conflicts:
#	Cargo.lock
craigwalkeruk pushed a commit to craigwalkeruk/custom-goose that referenced this pull request Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants