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

Disable PoolName validation #2459

Merged
merged 2 commits into from Sep 29, 2022
Merged

Disable PoolName validation #2459

merged 2 commits into from Sep 29, 2022

Conversation

Porges
Copy link
Member

@Porges Porges commented Sep 28, 2022

We are going to want some restrictions on pool names in future, but for now we are disabling validation to prevent breaking any existing users with non-conforming pool names.

@Porges Porges enabled auto-merge (squash) September 28, 2022 21:44
@codecov-commenter
Copy link

codecov-commenter commented Sep 28, 2022

Codecov Report

Merging #2459 (8f4cf9d) into main (8f4cf9d) will not change coverage.
The diff coverage is n/a.

❗ Current head 8f4cf9d differs from pull request most recent head 6bf39bc. Consider uploading reports for the commit 6bf39bc to get more accurate results

@@           Coverage Diff           @@
##             main    #2459   +/-   ##
=======================================
  Coverage   31.02%   31.02%           
=======================================
  Files         253      253           
  Lines       29023    29023           
=======================================
  Hits         9003     9003           
  Misses      20020    20020           

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

@Porges Porges force-pushed the disable-poolname-validation-2 branch from a16eae2 to 6bf39bc Compare September 29, 2022 04:11
@Porges Porges merged commit 1ec9b13 into main Sep 29, 2022
@Porges Porges deleted the disable-poolname-validation-2 branch October 10, 2022 19:45
@microsoft microsoft locked as resolved and limited conversation to collaborators Nov 10, 2022
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.

None yet

4 participants