Skip to content

docs: fix default projections setting discrepancy#51640

Merged
haampie merged 2 commits intodevelopfrom
copilot/fix-default-projections-docs
Nov 25, 2025
Merged

docs: fix default projections setting discrepancy#51640
haampie merged 2 commits intodevelopfrom
copilot/fix-default-projections-docs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 25, 2025 •
edited
Loading

The documentation in config_yaml.rst showed an incorrect default projection setting that didn't match the actual default in etc/spack/defaults/base/config.yaml.

Before (incorrect):

{architecture}/{compiler.name}-{compiler.version}/{name}-{version}-{hash}

After (matches actual default):

{architecture.platform}-{architecture.target}/{name}-{version}-{hash}
Original prompt

This section details on the original issue you should resolve

<issue_title>docs: discrepancy for default projections setting</issue_title>
<issue_description>There is a discrepancy in the docs for the default projections setting. This line specifying the default setting:

https://github.com/spack/spack/blob/develop/lib/spack/docs/config_yaml.rst?plain=1#L41

{architecture}/{compiler.name}-{compiler.version}/{name}-{version}-{hash}

compared to the setting in the default config.yaml:

https://github.com/spack/spack/blob/develop/etc/spack/defaults/base/config.yaml#L22

{architecture.platform}-{architecture.target}/{name}-{version}-{hash}

I assume that line in the docs was overlooked and should be replaced with the setting found in the default config.yaml.</issue_description>

<agent_instructions>The default config is in etc/spack/defaults/base/config.yaml (config -> install_tree -> root -> projections -> all).

Notice that there are also other types of projections related to modules and views, we're only talking about "install_tree" related projections.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix discrepancy in default projections documentation docs: fix default projections setting discrepancy Nov 25, 2025
Copilot AI requested a review from haampie November 25, 2025 10:52
@haampie haampie marked this pull request as ready for review November 25, 2025 10:56
@haampie haampie added v1.0.3 PRs to backport for v1.0.3 v1.1.1 labels Nov 25, 2025
@haampie haampie enabled auto-merge (squash) November 25, 2025 11:10
@haampie haampie merged commit c63f1cc into develop Nov 25, 2025
39 of 40 checks passed
@haampie haampie deleted the copilot/fix-default-projections-docs branch November 25, 2025 14:08
psakievich pushed a commit to psakievich/spack that referenced this pull request Dec 1, 2025
@haampie @psakievich
docs: fix default projections setting discrepancy (spack#51640)
274bcb8 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
hippo91 pushed a commit to hippo91/spack that referenced this pull request Dec 1, 2025
@haampie @hippo91
docs: fix default projections setting discrepancy (spack#51640)
8021194 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
becker33 pushed a commit that referenced this pull request Jan 10, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
de3a3c4 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
@becker33 becker33 mentioned this pull request Jan 10, 2026
Merged
2 tasks
becker33 pushed a commit that referenced this pull request Jan 11, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
20fceb1 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
becker33 pushed a commit that referenced this pull request Jan 15, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
423f5d1 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
vjranagit pushed a commit to vjranagit/spack that referenced this pull request Jan 18, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (spack#51640)
c69874b 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
becker33 pushed a commit that referenced this pull request Jan 22, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
5e569e6 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
becker33 pushed a commit that referenced this pull request Jan 26, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
e9bfade 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
@becker33 becker33 mentioned this pull request Feb 2, 2026
Merged
becker33 pushed a commit that referenced this pull request Feb 2, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
cb9aa2c 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
becker33 pushed a commit that referenced this pull request Feb 2, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
226d9e6 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
becker33 pushed a commit that referenced this pull request Feb 19, 2026
@haampie @becker33
docs: fix default projections setting discrepancy (#51640)
1523038 
Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: Harmen Stoppels <[email protected]>
Signed-off-by: Gregory Becker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kwryankrattiger kwryankrattiger kwryankrattiger approved these changes

+1 more reviewer

@haampie haampie haampie approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@haampie haampie

Copilot code review Copilot

Labels

v1.0.3 PRs to backport for v1.0.3 v1.1.1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

docs: discrepancy for default projections setting

3 participants

@haampie @kwryankrattiger