Link to https://aka.ms/bicep/core-diagnostics from core compiler diagnostics #14842
Labels
Milestone
Comments
anthony-c-martin
added a commit
that referenced
this issue
Aug 23, 2024
…tics (#14875) The split class structure between `ErrorDiagnostic` & `Diagnostic` is something I've been meaning to remove since the very early days of Bicep. I've decided to do it now, because it'll simplify the refactor needed for #14842. The majority of the changes in this PR were made using refactoring tools, not manually. The main changes I made were: * Removed `ErrorDiagnostic`, `ErrorBuilderDelegate`, `FixableErrorDiagnostic` * Converted `diag is ErrorDiagnostic` type checks to use `diag.Level == DiagnosticLevel.Error` instead * Renamed `ResultWithDiagnostic` to `ResultWithDiagnosticBuilder` instead to more accurately represent its purpose * Created `ResultWithDiagnotic<T>`, and replaced usage of `Result<T, Diagnostic>` with it * Other changes were mostly downstream changes of the above ###### Microsoft Reviewers: [Open in CodeFlow](https://microsoft.github.io/open-pr/?codeflow=https://github.com/Azure/bicep/pull/14875)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We now have a docs page for compiler errors!
We should link to it using the error code as an anchor in the URL:
https://aka.ms/bicep/core-diagnostics#{error_code}E.g.:
https://aka.ms/bicep/core-diagnostics#BCP050The text was updated successfully, but these errors were encountered: