Skip to content

#1087 changed default value of "execute-applied-concepts" from false to true default value - #1338

Merged
DirkMahler merged 2 commits into
masterfrom
1087-config-execute-applied-concepts-should-be-enabled-by-default
Aug 24, 2026
Merged

#1087 changed default value of "execute-applied-concepts" from false to true default value#1338
DirkMahler merged 2 commits into
masterfrom
1087-config-execute-applied-concepts-should-be-enabled-by-default

Conversation

@DirkMahler

Copy link
Copy Markdown
Member

No description provided.

@DirkMahler DirkMahler linked an issue Aug 21, 2026 that may be closed by this pull request

scanClasses(Assertions4Junit4.class, Assertions4Junit5.class, AbstractAssertions4Junit5.class);

// Execute the required concept explicitly

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This doesn't work anymore as the concepts will be re-applied. Disabled the tests in pom.xml

assertThat(conceptResult.getRows()).hasSize(4);

store.beginTransaction();
assertDisabledElements(conceptResult.getRows().stream()

@DirkMahler DirkMahler Aug 22, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Applied formatter to this class, therefore lots of changes, the relevant is commented below

@DirkMahler
DirkMahler requested review from cl90 and morpfl August 22, 2026 13:25

@cl90 cl90 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The second commit message refers to a different issue, so I'd recommend squashing both commits to fix it.

@DirkMahler
DirkMahler merged commit 61f42b8 into master Aug 24, 2026
9 checks passed
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.

Config execute-applied-concepts should be enabled by default

2 participants