Skip to content

chore(maintenance): make GitHub repo configurable for auto-updater and publisher#6828

Merged
jamadeo merged 1 commit intoblock:mainfrom
Yorisoft:yorisoft/fix_hardcoded_repo_path_in_updater
Feb 2, 2026
Merged

chore(maintenance): make GitHub repo configurable for auto-updater and publisher#6828
jamadeo merged 1 commit intoblock:mainfrom
Yorisoft:yorisoft/fix_hardcoded_repo_path_in_updater

Conversation

@Yorisoft
Copy link
Contributor

Summary

Based on my understanding from related issue and AI assistance:
The auto-updated has hard-coded string for the repo path. This is an issue because it would cause friction would repo migration.

This PR:
Replaces hard-coded 'block/goose' repository path with environment
variables (GITHUB_OWNER, GITHUB_REPO) that default to 'block/goose'.

Type of Change

  • Feature
  • Bug fix
  • Refactor / Code quality
  • Performance improvement
  • Documentation
  • Tests
  • Security fix
  • Build / Release
  • Other (specify below)
    Chore ?

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

cargo check \
&& cargo test \
&& cargo fmt \
&& ./scripts/clippy-lint.sh

Related Issues

Relates to #6136

…isher

Replace hard-coded 'block/goose' repository path with environment
variables (GITHUB_OWNER, GITHUB_REPO) that default to 'block/goose'.

Signed-off-by: Yelsin Sepulveda <yelsinsepulveda@gmail.com>
@jamadeo jamadeo self-requested a review February 2, 2026 20:59
@jamadeo jamadeo added this pull request to the merge queue Feb 2, 2026
Merged via the queue into block:main with commit fafda07 Feb 2, 2026
17 checks passed
katzdave added a commit that referenced this pull request Feb 3, 2026
…ntext

* 'main' of github.com:block/goose:
  refactor(providers): extract ProviderDef trait and OpenAiCompatibleProvider (#6832)
  feat: ask ai discord bot (#6842)
  chore(maintenance): make GitHub repo configurable for auto-updater and publisher (#6828)
  fix: make apps work in built copies of goose (#6901)
  Remove dependency on goose-mcp from goose crate (#6637)
  Clean up build canonical warnings (#6880)
  Sync desktop_prompt with UI (#6898)
stebbins pushed a commit to stebbins/goose that referenced this pull request Feb 4, 2026
…d publisher (block#6828)

Signed-off-by: Yelsin Sepulveda <yelsinsepulveda@gmail.com>
Signed-off-by: Harrison <hcstebbins@gmail.com>
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.

2 participants