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

Fix missed topic names non-conforming to Azure policy #426

Merged
merged 1 commit into from
Mar 5, 2024

Conversation

unjello
Copy link
Contributor

@unjello unjello commented Mar 5, 2024

No description provided.

@unjello unjello changed the title Fix missed topic names non-conforming to Azure policty Fix missed topic names non-conforming to Azure policy Mar 5, 2024
@unjello
Copy link
Contributor Author

unjello commented Mar 5, 2024

Sorry folks for a mess, but I missed couple changes outside of function :/

Copy link

codecov bot commented Mar 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.56%. Comparing base (c0aecac) to head (5faa83f).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #426      +/-   ##
==========================================
- Coverage   66.73%   66.56%   -0.17%     
==========================================
  Files          63       63              
  Lines        4139     4139              
==========================================
- Hits         2762     2755       -7     
- Misses       1213     1219       +6     
- Partials      164      165       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@m110
Copy link
Member

m110 commented Mar 5, 2024

Thanks @unjello!

@m110 m110 merged commit 9d4f783 into ThreeDotsLabs:master Mar 5, 2024
9 of 10 checks passed
# 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