-
Notifications
You must be signed in to change notification settings - Fork 121
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
Bug 1742448 - Allow passing in the build date #1917
Conversation
942529e
to
f1829e9
Compare
a0cd60e
to
64e851b
Compare
9eb58ab
to
70409b7
Compare
Adding multiple reviewers, whoever gets here first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks good, with just a couple of comments about things pointing to your personal GitHub. No reason to block on this as long as we have a followup bug to fix it.
gradle-plugin/src/main/groovy/mozilla/telemetry/glean-gradle-plugin/GleanGradlePlugin.groovy
Outdated
Show resolved
Hide resolved
Previously a second invocation of Glean.initialize would override the stored build info, even if no other work was done due to Glean already being initialized.
70409b7
to
78106d2
Compare
Codecov Report
@@ Coverage Diff @@
## main #1917 +/- ##
==========================================
- Coverage 30.30% 29.41% -0.90%
==========================================
Files 1 1
Lines 33 34 +1
==========================================
Hits 10 10
- Misses 23 24 +1
Continue to review full report at Codecov.
|
Drop in coverage is expected. We only record coverage from Rust tests and currently the build date is not used from within Rust |
Requires mozilla/glean_parser#431 & a release.
Requires mozilla-services/mozilla-pipeline-schemas#715