Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Rename EventGrid subscription #2960

Merged
merged 1 commit into from
Apr 4, 2023
Merged

Rename EventGrid subscription #2960

merged 1 commit into from
Apr 4, 2023

Conversation

Porges
Copy link
Member

@Porges Porges commented Apr 3, 2023

The EventGrid subscription name is invalid according to the published schema. Rename it, and add a deployment step to delete any old instances of this subscription.

@codecov-commenter
Copy link

codecov-commenter commented Apr 3, 2023

Codecov Report

Merging #2960 (341e189) into main (3c3f12a) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2960      +/-   ##
==========================================
- Coverage   29.09%   29.07%   -0.02%     
==========================================
  Files         303      304       +1     
  Lines       36170    36335     +165     
==========================================
+ Hits        10523    10566      +43     
- Misses      25647    25769     +122     

see 16 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Porges
Copy link
Member Author

Porges commented Apr 4, 2023

Deployment works fine

@Porges Porges enabled auto-merge (squash) April 4, 2023 02:30
@Porges Porges merged commit 34b513e into main Apr 4, 2023
@Porges Porges deleted the eventgrid-rename branch April 4, 2023 02:45
This was referenced Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants