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 1738878 - Glean parser with tags #1852

Merged
merged 2 commits into from
Nov 3, 2021
Merged

Bug 1738878 - Glean parser with tags #1852

merged 2 commits into from
Nov 3, 2021

Conversation

badboy
Copy link
Member

@badboy badboy commented Nov 3, 2021

Requires mozilla/glean_parser#409 and a followup release.
We should probably document that.

cc @wlach

This brings tags support
@auto-assign auto-assign bot requested a review from travis79 November 3, 2021 10:13
@badboy badboy force-pushed the glean-parser-with-tags branch from cbf1bd5 to 31ac451 Compare November 3, 2021 10:14
Also adds tags to the sample app to test it out.
@badboy badboy force-pushed the glean-parser-with-tags branch from 31ac451 to 3ed4206 Compare November 3, 2021 14:34
Copy link
Contributor

@Dexterp37 Dexterp37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems sane to me!

@badboy
Copy link
Member Author

badboy commented Nov 3, 2021

Test is green now (but running with SSH enabled, I'll cancel that, then still merge this)

@badboy badboy merged commit 0404d27 into main Nov 3, 2021
@badboy badboy deleted the glean-parser-with-tags branch November 3, 2021 14:52
# 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