Skip to content

docs(prepare): add all source files to sources#7498

Merged
jdx merged 2 commits into
jdx:mainfrom
risu729:prepare-sources-docs
Dec 28, 2025
Merged

docs(prepare): add all source files to sources#7498
jdx merged 2 commits into
jdx:mainfrom
risu729:prepare-sources-docs

Conversation

@risu729
Copy link
Copy Markdown
Contributor

@risu729 risu729 commented Dec 28, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 28, 2025 16:59
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 updates the documentation table for built-in package manager providers to show all source files (not just lockfiles) that each provider monitors. The table headers are updated from "Lockfile" to "Sources" and "Output" to "Outputs" to better reflect the complete set of files that trigger prepare operations.

Key Changes:

  • Updated table column headers to "Sources" (from "Lockfile") and "Outputs" (from "Output")
  • Added package manifest files (package.json, pyproject.toml, Gemfile, composer.json) alongside their respective lockfiles in the Sources column
  • Maintained accurate output directories and commands for each provider

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

Comment thread docs/dev-tools/prepare.md
| `bundler` | `Gemfile`, `Gemfile.lock` | `vendor/bundle/` | `bundle install` |
| `composer` | `composer.json`, `composer.lock` | `vendor/` | `composer install` |

Built-in providers are only active when explicitly configured in `mise.toml` and their lockfile exists.
Copy link

Copilot AI Dec 28, 2025

Choose a reason for hiding this comment

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

This sentence should be updated to reflect the new "Sources" terminology used in the table above. Since the table now includes all source files (not just lockfiles), this should say "source files" instead of "lockfile". For example: "Built-in providers are only active when explicitly configured in mise.toml and their source files exist."

Suggested change
Built-in providers are only active when explicitly configured in `mise.toml` and their lockfile exists.
Built-in providers are only active when explicitly configured in `mise.toml` and their source files exist.

Copilot uses AI. Check for mistakes.
@jdx jdx merged commit d943c0f into jdx:main Dec 28, 2025
33 checks passed
@risu729 risu729 deleted the prepare-sources-docs branch December 29, 2025 00:45
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.

3 participants