Skip to content

Remove deprecated config options#44061

Merged
haampie merged 1 commit intospack:developfrom
alalazo:removal/remove-deprecated-config-options
Sep 6, 2024
Merged

Remove deprecated config options#44061
haampie merged 1 commit intospack:developfrom
alalazo:removal/remove-deprecated-config-options

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented May 7, 2024

These options have been deprecated in v0.21, and slated for removal in v0.22

@alalazo alalazo added this to the v0.22.0 milestone May 7, 2024
@spackbot-app spackbot-app bot added commands core PR affects Spack core functionality tests General test capability(ies) labels May 7, 2024
@alalazo alalazo mentioned this pull request May 7, 2024
7 tasks
@haampie
Copy link
Copy Markdown
Member

haampie commented May 7, 2024

let's wait a little bit...

@alalazo
Copy link
Copy Markdown
Member Author

alalazo commented May 7, 2024

If we wanna wait, we need to update the message.

@tgamblin
Copy link
Copy Markdown
Member

tgamblin commented May 7, 2024

Let's move the deprecation rate to 2 releases based on feedback from @haampie that users generally don't want to change this quickly. After 1.0 we should probably go even slower than that (if we remove options at all)

@alalazo alalazo modified the milestones: v0.22.0, v0.23.0 May 7, 2024
These options have been deprecated in v0.21, and
slated for removal in v0.23
@alalazo alalazo force-pushed the removal/remove-deprecated-config-options branch from 25bb293 to d486d83 Compare September 6, 2024 07:31
@haampie haampie enabled auto-merge (squash) September 6, 2024 10:34
@haampie haampie merged commit a3fa548 into spack:develop Sep 6, 2024
@alalazo alalazo deleted the removal/remove-deprecated-config-options branch September 7, 2024 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commands core PR affects Spack core functionality tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants