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

Add the tmp directory to .gitignore #6349

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

evan-bradley
Copy link
Contributor

Running make genjsonschema creates a tmp directory that is not present in the .gitignore file, this makes sure contributors don't need to worry about having it in their repo directory.

Running `make genjsonschema` creates a `tmp` directory that is not added to the .gitignore file.
@evan-bradley evan-bradley requested a review from a team as a code owner November 19, 2024 20:25
@evan-bradley
Copy link
Contributor Author

This PR will need a Skip Changelog tag.

Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.9%. Comparing base (35a3157) to head (02fcabd).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #6349   +/-   ##
=====================================
  Coverage   66.9%   66.9%           
=====================================
  Files        193     193           
  Lines      15652   15652           
=====================================
  Hits       10479   10479           
  Misses      4882    4882           
  Partials     291     291           
---- 🚨 Try these New Features:

@pellared pellared added the Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG label Nov 19, 2024
@pellared pellared merged commit 3704ded into open-telemetry:main Nov 20, 2024
24 checks passed
dmathieu added a commit that referenced this pull request Dec 3, 2024
#6347 is
updating the generated JSON schema, as it has changed with upgrades of
`go-jsonschema`.
We update the binary semi automatically with dependabot. However, this
means there could be unseen regressions or changes like this that we
miss.

This runs the schema generation on CI, before checking for a clean
repository, so we can keep the generated code up to date with the binary
version.

This should fail until #6347 and #6349 are merged.

---------

Co-authored-by: Robert Pająk <[email protected]>
@MrAlias MrAlias added this to the v1.33.0 milestone Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants