Skip to content

Conversation

@jkroepke
Copy link
Member

What this PR does / why we need it

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

  • fixes #

Special notes for your reviewer

Particularly user-facing changes

Checklist

Complete these before marking the PR as ready to review:

  • DCO signed
  • The PR title has a summary of the changes and the area they affect
  • The PR body has a summary to reflect any significant (and particularly user-facing) changes introduced by this PR

Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
@jkroepke jkroepke requested a review from a team as a code owner April 14, 2025 11:28
@jkroepke jkroepke requested review from Copilot and removed request for a team April 14, 2025 11:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

README.md:1

  • The header line appears to have an unintended prefix 'config.file'; please confirm if it should simply be '# windows_exporter'.
config.file# windows_exporter

README.md:94

  • Ensure that the removal of the '--config.file.insecure-skip-verify' option is consistently reflected across all relevant documentation and examples.
`--config.file.insecure-skip-verify`

@jkroepke jkroepke merged commit f1f763b into master Apr 14, 2025
10 of 11 checks passed
@jkroepke jkroepke deleted the jkroepke-patch-1 branch April 14, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants