Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[chore][pkg/windowsperfcounters] Fix lint on Windows #30671

Merged

Conversation

pjanotti
Copy link
Contributor

Description:
Fixing lint issues when GOOS=windows. This is in preparation to eventually include lint with GOOS=windows as part of CI.

Link to tracking Issue:
N/A

Testing:
make on component folder on a Windows box

Documentation:
N/A

@pjanotti
Copy link
Contributor Author

Please, add Run Windows label.

@songy23 songy23 added the Run Windows Enable running windows test on a PR label Jan 19, 2024
@djaglowski
Copy link
Member

CI is failing but should be resolved after #30640 is merged.

@djaglowski djaglowski merged commit 187c88a into open-telemetry:main Jan 22, 2024
92 of 93 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 22, 2024
@pjanotti pjanotti deleted the windows-lint-pkg-winperfcounters branch January 22, 2024 20:23
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
…30671)

**Description:**
Fixing lint issues when GOOS=windows. This is in preparation to
eventually include lint with GOOS=windows as part of CI.

**Link to tracking Issue:**
N/A

**Testing:**
`make` on component folder on a Windows box

**Documentation:**
N/A

Co-authored-by: Daniel Jaglowski <jaglows3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg/winperfcounters Run Windows Enable running windows test on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants