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] catch if a file was generated and not added to working tree #32730

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Apr 28, 2024

Description:
This changes the check for changes for the make generate step. We check if any files are present that are not checked in.

@atoulme atoulme requested a review from a team as a code owner April 28, 2024 22:08
@atoulme
Copy link
Contributor Author

atoulme commented Apr 29, 2024

Tests don't pass, proving this test is effective. Once #32731 is merged, please rebase to latest main and let's see if the tests pass now.

@evan-bradley evan-bradley merged commit 64645c1 into open-telemetry:main May 2, 2024
160 checks passed
@github-actions github-actions bot added this to the next release milestone May 2, 2024
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this pull request May 8, 2024
…pen-telemetry#32730)

**Description:**
This changes the check for changes for the `make generate` step. We
check if any files are present that are not checked in.

Co-authored-by: Evan Bradley <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jul 11, 2024
…pen-telemetry#32730)

**Description:**
This changes the check for changes for the `make generate` step. We
check if any files are present that are not checked in.

Co-authored-by: Evan Bradley <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
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.

None yet

4 participants