Skip to content

Clarify the use of multiple profile expressions with "spring.config.activate.on-profile"#44866

Closed
quaff wants to merge 1 commit intospring-projects:mainfrom
quaff:patch-147
Closed

Clarify the use of multiple profile expressions with "spring.config.activate.on-profile"#44866
quaff wants to merge 1 commit intospring-projects:mainfrom
quaff:patch-147

Conversation

@quaff
Copy link
Contributor

@quaff quaff commented Mar 25, 2025 •
edited
Loading

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 25, 2025
@wilkinsona
Copy link
Member

wilkinsona commented Mar 25, 2025

Thanks for the suggestion but I think this may cause confusion. , isn't really the equivalent of |. It's a separator for the expressions when configured as a single comma-separate value due to it being bound to a String[]. If it's configured as a list, no separation occurs. In other words, the following two snippets of YAML are not equivalent:

spring.config.activate.on-profile:
    - a|b
spring.config.activate.on-profile:
    - a,b

I think it would be better to update the table in this section of the documentation to describe on-profile more accurately. Perhaps something like "profile expressions of which at least one must match for the document to be active".

@wilkinsona wilkinsona added the status: waiting-for-feedback We need additional information before we can continue label Mar 25, 2025
@quaff
Copy link
Contributor Author

quaff commented Mar 26, 2025

Updated, could someone review the wording?

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Mar 26, 2025
@snicoll snicoll changed the title Clarify the comma in "spring.config.activate.on-profile" value and add test Clarify the use of multiple profile expressions with "spring.config.activate.on-profile" Apr 17, 2025
@snicoll snicoll added type: documentation A documentation update and removed status: waiting-for-triage An issue we've not yet triaged status: feedback-provided Feedback has been provided labels Apr 17, 2025
@snicoll snicoll self-assigned this Apr 17, 2025
@snicoll snicoll added this to the 3.3.11 milestone Apr 17, 2025
@snicoll snicoll mentioned this pull request Apr 17, 2025
Closed
snicoll pushed a commit that referenced this pull request Apr 17, 2025
@quaff @snicoll
Document the semantic of using multiple profile expressions
6eae09b 
This commit clarifies the semantic of declaring multiple profile
expression with the "spring.config.activate.on-profile" property.

See gh-44866

Signed-off-by: Yanming Zhou <zhouyanming@gmail.com>
snicoll added a commit that referenced this pull request Apr 17, 2025
@snicoll
Polish "Document the semantic of using multiple profile expressions"
6eb20ab 
See gh-44866
@snicoll snicoll closed this in ad61236 Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@snicoll snicoll

Labels

type: documentation A documentation update

Projects

None yet

Milestone

3.3.11

Development

Successfully merging this pull request may close these issues.

4 participants

@quaff @wilkinsona @snicoll @spring-projects-issues