Skip to content

docs: fix wrong file reference to forgejo backend implemenation#9090

Merged
jdx merged 1 commit intojdx:mainfrom
roele:issues/doc-fix-forgejo
Apr 14, 2026
Merged

docs: fix wrong file reference to forgejo backend implemenation#9090
jdx merged 1 commit intojdx:mainfrom
roele:issues/doc-fix-forgejo

Conversation

@roele
Copy link
Copy Markdown
Contributor

@roele roele commented Apr 14, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 14, 2026 19:50
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Forgejo backend documentation to reference the correct source file, changing the path from forgejo.rs to github.rs. A review comment suggests removing the ./ prefix from the file path to maintain consistency with other file references in the documentation.

Comment thread docs/dev-tools/backends/forgejo.md Outdated
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

This PR corrects the documentation link for the Forgejo backend implementation by pointing to the actual backend source file used in the codebase.

Changes:

  • Update Forgejo backend documentation to reference src/backend/github.rs (the unified Git backend used for GitHub/GitLab/Forgejo).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@roele roele force-pushed the issues/doc-fix-forgejo branch from e6c03b3 to 35a271a Compare April 14, 2026 19:53
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 14, 2026

Greptile Summary

This PR fixes two broken source file links in docs/dev-tools/backends/forgejo.md. The original doc referenced src/backend/forgejo.rs (which doesn't exist — the Forgejo backend is colocated with GitHub/GitLab logic in src/backend/github.rs) and src/backend/asset_detector.rs (which was renamed to src/backend/asset_matcher.rs). Both corrections are accurate and verified against the repository file structure.

Confidence Score: 5/5

Safe to merge — documentation-only fix correcting two broken source-file links, both verified against actual file paths in the repository.

Both corrections are factually accurate: src/backend/github.rs exists and contains all Forgejo dispatch logic, and src/backend/asset_matcher.rs is the real filename. No logic, configuration, or behavior is touched.

No files require special attention.

Important Files Changed

Filename Overview
docs/dev-tools/backends/forgejo.md Two broken source-file hyperlinks corrected: forgejo.rsgithub.rs (Forgejo backend lives in the shared GitHub/GitLab/Forgejo handler) and asset_detector.rsasset_matcher.rs (actual filename in the repo).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["docs/dev-tools/backends/forgejo.md"] -->|"links to (fixed)"| B["src/backend/github.rs\n(handles GitHub + GitLab + Forgejo)"]
    A -->|"links to (fixed)"| C["src/backend/asset_matcher.rs\n(shared autodetection logic)"]
    B -->|"is_forgejo() dispatch"| D["forgejo:: module\n(API calls, token resolution)"]
    C -->|"used by"| B
    C -->|"used by"| E["GitLab backend"]
Loading

Reviews (3): Last reviewed commit: "docs: fix wrong file reference to forgej..." | Re-trigger Greptile

@roele roele force-pushed the issues/doc-fix-forgejo branch from 35a271a to f3499c3 Compare April 14, 2026 20:03
@jdx jdx enabled auto-merge (squash) April 14, 2026 20:16
@jdx jdx merged commit 447bed4 into jdx:main Apr 14, 2026
35 checks passed
@roele roele deleted the issues/doc-fix-forgejo branch April 14, 2026 22:07
mise-en-dev added a commit that referenced this pull request Apr 15, 2026
### 🚀 Features

- **(npm)** use --min-release-age for npm 11.10.0+ supply chain
protection by @webkaz in [#9072](#9072)
- **(registry)** add openfga by @mnm364 in
[#9084](#9084)
- **(task)** allow to set confirmation default by @roele in
[#9089](#9089)
- support os/arch compound syntax in tool os filtering by @RobertDeRose
in [#9088](#9088)

### 🐛 Bug Fixes

- **(activate)** export __MISE_EXE and resolve bare ARGV0 to absolute
path by @fru1tworld in [#9081](#9081)
- **(install)** support aliased installs sharing a backend by @jdx in
[#9093](#9093)
- **(shim)** use which_no_shims when resolving mise binary in reshim and
doctor by @kevinswiber in [#9071](#9071)
- filter empty segments in colon-separated env var parsing by @baby-joel
in [#9076](#9076)

### 📚 Documentation

- fix wrong file reference to forgejo backend implemenation by @roele in
[#9090](#9090)
- fix cli token command for token resolution by @roele in
[#9077](#9077)

### 📦 Registry

- add trzsz-go
([aqua:trzsz/trzsz-go](https://github.com/trzsz/trzsz-go)) by
@ZeroAurora in [#9083](#9083)
- add copilot
([aqua:github/copilot-cli](https://github.com/github/copilot-cli)) by
@risu729 in [#9082](#9082)

### Chore

- add AGENTS.md symlink by @jdx in
[#9094](#9094)

### New Contributors

- @kevinswiber made their first contribution in
[#9071](#9071)
- @webkaz made their first contribution in
[#9072](#9072)
- @RobertDeRose made their first contribution in
[#9088](#9088)

## 📦 Aqua Registry Updates

#### New Packages (7)

-
[`IBM-Cloud/ibm-cloud-cli-release`](https://github.com/IBM-Cloud/ibm-cloud-cli-release)
- [`max-sixty/worktrunk`](https://github.com/max-sixty/worktrunk)
- [`micelio.dev/hif`](https://github.com/micelio.dev/hif)
- [`pgplex/pgschema`](https://github.com/pgplex/pgschema)
-
[`rose-pine/rose-pine-bloom`](https://github.com/rose-pine/rose-pine-bloom)
- [`santosr2/TerraTidy`](https://github.com/santosr2/TerraTidy)
- [`trzsz/trzsz-go`](https://github.com/trzsz/trzsz-go)

#### Updated Packages (3)

- [`mvdan/sh`](https://github.com/mvdan/sh)
- [`rvben/rumdl`](https://github.com/rvben/rumdl)
- [`temporalio/temporal`](https://github.com/temporalio/temporal)
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Apr 16, 2026
## [2026.4.14](https://github.com/jdx/mise/compare/v2026.4.13..v2026.4.14) - 2026-04-15

### Chore

- bump sigstore-verification by @jdx in [#9128](jdx/mise#9128)

## [2026.4.13](https://github.com/jdx/mise/compare/v2026.4.12..v2026.4.13) - 2026-04-15

### 🐛 Bug Fixes

- **(go)** honor install_before for module versions by @mariusvniekerk in [#9097](jdx/mise#9097)
- **(vfox-plugin)** support Git URL with commit hash for mise.toml by @Oyami-Srk in [#9099](jdx/mise#9099)
- `MISE_FETCH_REMOTE_VERSIONS_CACHE` not respected by @mcncl in [#9096](jdx/mise#9096)

### 📦️ Dependency Updates

- unblock cargo-deny advisories check by @jdx in [#9112](jdx/mise#9112)

### New Contributors

- @mariusvniekerk made their first contribution in [#9097](jdx/mise#9097)
- @mcncl made their first contribution in [#9096](jdx/mise#9096)
- @Oyami-Srk made their first contribution in [#9099](jdx/mise#9099)

## [2026.4.12](https://github.com/jdx/mise/compare/v2026.4.11..v2026.4.12) - 2026-04-15

### 🚀 Features

- **(npm)** use --min-release-age for npm 11.10.0+ supply chain protection by @webkaz in [#9072](jdx/mise#9072)
- **(registry)** add openfga by @mnm364 in [#9084](jdx/mise#9084)
- **(task)** allow to set confirmation default by @roele in [#9089](jdx/mise#9089)
- support os/arch compound syntax in tool os filtering by @RobertDeRose in [#9088](jdx/mise#9088)

### 🐛 Bug Fixes

- **(activate)** export __MISE_EXE and resolve bare ARGV0 to absolute path by @fru1tworld in [#9081](jdx/mise#9081)
- **(install)** support aliased installs sharing a backend by @jdx in [#9093](jdx/mise#9093)
- **(shim)** use which_no_shims when resolving mise binary in reshim and doctor by @kevinswiber in [#9071](jdx/mise#9071)
- filter empty segments in colon-separated env var parsing by @baby-joel in [#9076](jdx/mise#9076)

### 📚 Documentation

- fix wrong file reference to forgejo backend implemenation by @roele in [#9090](jdx/mise#9090)
- fix cli token command for token resolution by @roele in [#9077](jdx/mise#9077)
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