ref(ownership): remove premunge option #77334
Conversation
github-actions
bot
added
the
Scope: Backend
| @@ -141,8 +141,8 @@ def test_with_munged( | |||
| self, data: PathSearchable, munged_data: tuple[Sequence[Mapping[str, Any]], Sequence[str]] | |||
There was a problem hiding this comment.
I think this can this just fully replace test now?
There was a problem hiding this comment.
yes, need to rewrite a bunch of tests. can update this PR to do so
There was a problem hiding this comment.
👍 definitely doesn't need to happen with this PR specifically in my opinion if you want to get this out sooner
|
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This issue has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you remove the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
our test of improved ownership evaluation with only munging data once was a success https://app.datadoghq.com/metric/explorer?fromUser=false&start=1726079473808&end=1726080373808&paused=false#N4Ig7glgJg5gpgFxALlAGwIYE8D2BXJVEADxQEYAaELcqyKBAC1pEbghkcLIF8qo4AMwgA7CAgg4RKUAiwAHOChASAtnADOcAE4RNIKtrgBHPJoQaUAbVBGN8qVoD6gnNtUZCKiOq279VKY6epbINiAiGOrKQdpYZAYgUJ4YThr42gDGSsgg6gi6mZaBZnHKHsTIWiIFWAB0DhoITvLaONkaGnWMGCJQaHBOOGAiOk4YnRwi6jV1FcAAVDx1bWhoePIAFBUAlCA8ALpUru54mKHhJ6pnGDGl8ftHIE1YAzIg8hgDCAg5STgwJyZc4aCCZRJoUSDOSKZTpSFQCFQpz0JjKERuDxofb8CD2TBYJwwv6Q0aPHh8Z7ySEIADCUmEMBQIjOaB4QA
this PR removes the option and uses it by default.