You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ijemmy opened this issue
Jan 6, 2022
· 1 comment
· Fixed by #424
Assignees
Labels
completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)metricsThis item relates to the Metrics Utility
In #377 , we renamed the function from purgeStoredMetrics() to publishStoredMetrics(). We missed one reference in the CDK example project. Also, we still use the term purge in one of the unit tests.
The text was updated successfully, but these errors were encountered:
ijemmy
added
bug
Something isn't working
triage
This item has not been triaged by a maintainer, please wait
labels
Jan 6, 2022
ijemmy
changed the title
fix(examples): Remove purgeStoredMetrics() function usage in examples/cdk folder
fix(metrics): Rename purgeStoredMetrics() function usage in CDK example
Jan 6, 2022
dreamorosi
added
internal
PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)
metrics
This item relates to the Metrics Utility
and removed
triage
This item has not been triaged by a maintainer, please wait
labels
Jan 6, 2022
Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.
dreamorosi
changed the title
fix(metrics): Rename purgeStoredMetrics() function usage in CDK example
Docs: Rename purgeStoredMetrics() function usage in CDK example
Nov 14, 2022
completedThis item is complete and has been merged/shippedinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)metricsThis item relates to the Metrics Utility
Bug description
In #377 , we renamed the function from
purgeStoredMetrics()
topublishStoredMetrics()
. We missed one reference in the CDK example project. Also, we still use the termpurge
in one of the unit tests.Expected Behavior
There should be no compilation issue
Current Behavior
There's a compilation issue from
purgeStoredMetrics()
usagePossible Solution
Change from "purge" to "publish"
Steps to Reproduce
N/A
Environment
Related issues, RFCs
#377
The text was updated successfully, but these errors were encountered: