Skip to content
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

bump bsl-parser and Disable diagnostics #2869

Merged
merged 9 commits into from
Oct 14, 2022
Merged

Conversation

asosnoviy
Copy link
Member

@asosnoviy asosnoviy commented Aug 5, 2022

@disable для диагностик
Поднятие версии парсера
Выключение диагностики FieldsFromJoinsWithoutIsNull

@artbear
Copy link
Contributor

artbear commented Sep 25, 2022

Интересно, падает тест
FieldsFromJoinsWithoutIsNullDiagnosticTest > test() FAILED
java.lang.AssertionError at FieldsFromJoinsWithoutIsNullDiagnosticTest.java:124

@asosnoviy asosnoviy force-pushed the feature/bsl-parser branch 5 times, most recently from c57b775 to 9e3901a Compare October 12, 2022 08:26
@asosnoviy asosnoviy changed the title bump bsl-parser bump bsl-parser and Disable diagnostics Oct 12, 2022
@asosnoviy asosnoviy requested a review from nixel2007 October 12, 2022 09:08
@@ -81,6 +81,7 @@ public List<BSLDiagnostic> diagnostics(DocumentContext documentContext) {
.filter(info -> correctModuleType(info, moduleType, fileType))
.filter(info -> passedCompatibilityMode(info, compatibilityMode))
.map(DiagnosticInfo::getDiagnosticClass)
.filter(diagnostic -> !diagnostic.isAnnotationPresent(Disabled.class))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не стрельнет в случае декоратора? может взять сприновый AnnotationUtils, который проверит иерархию классов?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я не справился с AnnotationUtils

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell E 4 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@nixel2007 nixel2007 merged commit b0490a9 into develop Oct 14, 2022
@nixel2007 nixel2007 deleted the feature/bsl-parser branch October 14, 2022 19:58
@nixel2007 nixel2007 added this to the 0.21.0 milestone Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants