Skip to content

refactor(core): normalized boolean config resolution#2433

Merged
aryamohanan merged 6 commits into
fix-config-precedencefrom
fix-config-refactor-1
Mar 27, 2026
Merged

refactor(core): normalized boolean config resolution#2433
aryamohanan merged 6 commits into
fix-config-precedencefrom
fix-config-refactor-1

Conversation

@aryamohanan
Copy link
Copy Markdown
Contributor

@aryamohanan aryamohanan commented Mar 26, 2026

Normalized boolean config

refs https://jsw.ibm.com/browse/INSTA-82698

@aryamohanan aryamohanan requested a review from a team as a code owner March 26, 2026 14:05
Comment thread packages/core/src/config/index.js
Comment thread packages/core/src/config/index.js
@sonarqubecloud
Copy link
Copy Markdown

expect(result).to.equal(false);
});

it.skip('should prioritize env var over config value', () => {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

skipped precdence test for now, enable this in later PR's

@aryamohanan aryamohanan changed the title refactor(core): normalized boolean config resolution with env precedence refactor(core): normalized boolean config resolution Mar 26, 2026
});

it('should prioritize env var over config value', () => {
it.skip('should prioritize env var over config value', () => {
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.

I guess its skipped because it fails currently?
Maybe put a TODO to not forget about it :)

Copy link
Copy Markdown
Contributor

@kirrg001 kirrg001 left a comment

Choose a reason for hiding this comment

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

Great direction!

@aryamohanan aryamohanan merged commit 1708f46 into fix-config-precedence Mar 27, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants