Skip to content

refactor: standardized config logging format#2446

Merged
abhilash-sivan merged 2 commits into
fix-config-precedencefrom
fix-config-logging
Apr 6, 2026
Merged

refactor: standardized config logging format#2446
abhilash-sivan merged 2 commits into
fix-config-precedencefrom
fix-config-logging

Conversation

@aryamohanan
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread packages/core/src/config/index.js Outdated
config.packageJsonPath = process.env.INSTANA_PACKAGE_JSON_PATH;
logger.debug(`[config] env:INSTANA_PACKAGE_JSON_PATH = ${process.env.INSTANA_PACKAGE_JSON_PATH}`);
} else if (config.packageJsonPath != null && typeof config.packageJsonPath === 'string') {
logger.debug(`[config] config:config.packageJsonPath = ${config.packageJsonPath}`);
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.

How about we use the prefix "incode"?

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.

Yeah I am +1 for this. updated

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.

Only one suggestion. Pre-approving

@abhilash-sivan abhilash-sivan marked this pull request as ready for review April 6, 2026 10:46
@abhilash-sivan abhilash-sivan requested a review from a team as a code owner April 6, 2026 10:46
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 6, 2026

@abhilash-sivan abhilash-sivan merged commit 1375828 into fix-config-precedence Apr 6, 2026
3 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.

3 participants