Skip to content
New issue

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

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

Already on GitHub? # to your account

Bug 1933542 - Require array or object as the root type in object metrics #780

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

badboy
Copy link
Member

@badboy badboy commented Dec 23, 2024

Pull Request checklist

  • Quality: This PR builds and tests run cleanly
    • make test runs without emitting any warnings
    • make lint runs without emitting any errors
  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Changelog: This PR includes a changelog entry to CHANGELOG.md or an explanation of why it does not need one
    • Any breaking changes to language binding APIs are noted explicitly

@badboy badboy requested a review from a team as a code owner December 23, 2024 13:45
@badboy badboy requested review from chutten and removed request for a team December 23, 2024 13:45
While this could be considered breaking, it's not in practice.
E.g. the FOG code implementing object metrics already requires arrays/objects at the root.
@badboy badboy force-pushed the 1933542/object-metric-allow-only-nested branch from 0e5dca0 to 2819239 Compare December 23, 2024 13:46
@badboy badboy requested a review from travis79 January 6, 2025 12:57
@badboy badboy enabled auto-merge (rebase) January 6, 2025 12:57
@badboy badboy merged commit 604d07d into main Jan 6, 2025
8 checks passed
@badboy badboy deleted the 1933542/object-metric-allow-only-nested branch January 7, 2025 10:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants