-
Notifications
You must be signed in to change notification settings - Fork 764
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improved error message on existing resource properties #1881
Improved error message on existing resource properties #1881
Conversation
f991267
to
5eedf2b
Compare
Codecov Report
@@ Coverage Diff @@
## main #1881 +/- ##
==========================================
- Coverage 95.11% 95.11% -0.01%
==========================================
Files 371 371
Lines 21962 21969 +7
Branches 15 15
==========================================
+ Hits 20890 20896 +6
- Misses 1072 1073 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -1501,3 +1501,9 @@ resource vnet 'Microsoft.Network/virtualNetworks@2020-06-01' = { | |||
} | |||
} | |||
|
|||
resource existngResProperty 'Mock.Rp/mockExistingResource@2020-01-01' = { | |||
//@[28:69) [BCP081 (Warning)] Resource type "Mock.Rp/mockExistingResource@2020-01-01" does not have types available. |'Mock.Rp/mockExistingResource@2020-01-01'| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BCP081 [](start = 12, length = 6)
This isn't the diagnostic that was added (BCP167) Can probably change it to a real resource type to get that one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment.
…sources-error-message-1507
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great!
Fixes #1507
Contributing a Pull Request
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check. Once you are done and ready to submit your PR, run through the relevant checklist below.
Contributing a feature