You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by le...@apache.org on 2019/04/19 13:20:17 UTC
[incubator-druid] branch master updated: Introduce Non-TeamCity
Warning inspection severity in IntelliJ's profile (#7499)
This is an automated email from the ASF dual-hosted git repository.
leventov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git
The following commit(s) were added to refs/heads/master by this push:
new 0a0fd63 Introduce Non-TeamCity Warning inspection severity in IntelliJ's profile (#7499)
0a0fd63 is described below
commit 0a0fd63f4ed0783901bff80fbc103e99a6326aa7
Author: Roman Leventov <le...@gmail.com>
AuthorDate: Fri Apr 19 15:20:08 2019 +0200
Introduce Non-TeamCity Warning inspection severity in IntelliJ's profile (#7499)
In the IDE interface, "Non-TeamCity Warning" looks exactly like an ordinary warning, but TeamCity should be unaware of it.
This may help to workaround these issues: https://youtrack.jetbrains.com/issue/IDEA-209789 and https://youtrack.jetbrains.com/issue/IDEA-209791, that block the upgrade of IntelliJ engine used in the TeamCity build. It seems like there may be a bug that leads to false positive error and the build fail in this PR: https://github.com/apache/incubator-druid/pull/6702.
Removed the comment regarding "StaticPseudoFunctionalStyleMethod" inspection because the IntelliJ keeps removing it, see this issue: https://youtrack.jetbrains.com/issue/IDEA-211087
---
.idea/inspectionProfiles/Druid.xml | 6 ++----
.idea/inspectionProfiles/profiles_settings.xml | 20 ++++++++++++++++++++
2 files changed, 22 insertions(+), 4 deletions(-)
diff --git a/.idea/inspectionProfiles/Druid.xml b/.idea/inspectionProfiles/Druid.xml
index 9bc1a75..0a9adb1 100644
--- a/.idea/inspectionProfiles/Druid.xml
+++ b/.idea/inspectionProfiles/Druid.xml
@@ -306,8 +306,6 @@
</inspection_tool>
<inspection_tool class="StaticCallOnSubclass" enabled="true" level="ERROR" enabled_by_default="true" />
<inspection_tool class="StaticFieldReferenceOnSubclass" enabled="true" level="ERROR" enabled_by_default="true" />
- <!-- The current rate of false-positives produced by StaticPseudoFunctionalStyleMethod is very high,
- see https://youtrack.jetbrains.com/issue/IDEA-153047#focus=streamItem-27-3326648.0-0 -->
<inspection_tool class="StaticPseudoFunctionalStyleMethod" enabled="true" level="INFORMATION" enabled_by_default="true" />
<inspection_tool class="StringConcatenationInFormatCall" enabled="true" level="ERROR" enabled_by_default="true" />
<inspection_tool class="StringConcatenationInMessageFormatCall" enabled="true" level="ERROR" enabled_by_default="true" />
@@ -332,7 +330,7 @@
</inspection_tool>
<inspection_tool class="SuspiciousSystemArraycopy" enabled="true" level="ERROR" enabled_by_default="true" />
<inspection_tool class="SuspiciousToArrayCall" enabled="true" level="ERROR" enabled_by_default="true" />
- <inspection_tool class="SyntaxError" enabled="true" level="WARNING" enabled_by_default="true" />
+ <inspection_tool class="SyntaxError" enabled="true" level="Non-TeamCity Warning" enabled_by_default="true" />
<inspection_tool class="TextLabelInSwitchStatement" enabled="true" level="ERROR" enabled_by_default="true" />
<inspection_tool class="ThrowableNotThrown" enabled="true" level="ERROR" enabled_by_default="true" />
<inspection_tool class="ToArrayCallWithZeroLengthArrayArgument" enabled="true" level="WARNING" enabled_by_default="true">
@@ -363,7 +361,7 @@
<inspection_tool class="UnusedLibrary" enabled="true" level="WARNING" enabled_by_default="true" />
<inspection_tool class="UseOfPropertiesAsHashtable" enabled="true" level="WARNING" enabled_by_default="true" />
<inspection_tool class="VariableNotUsedInsideIf" enabled="true" level="WARNING" enabled_by_default="true" />
- <inspection_tool class="XmlHighlighting" enabled="true" level="WARNING" enabled_by_default="true" />
+ <inspection_tool class="XmlHighlighting" enabled="true" level="Non-TeamCity Warning" enabled_by_default="true" />
<inspection_tool class="unused" enabled="true" level="WARNING" enabled_by_default="true">
<scope name="UnusedInspectionsScope" level="ERROR" enabled="true">
<option name="LOCAL_VARIABLE" value="true" />
diff --git a/.idea/inspectionProfiles/profiles_settings.xml b/.idea/inspectionProfiles/profiles_settings.xml
index 67c3084..75b0a83 100644
--- a/.idea/inspectionProfiles/profiles_settings.xml
+++ b/.idea/inspectionProfiles/profiles_settings.xml
@@ -2,5 +2,25 @@
<settings>
<option name="PROJECT_PROFILE" value="Druid" />
<version value="1.0" />
+ <info color="be9117">
+ <option name="BACKGROUND" value="52503a" />
+ <option name="ERROR_STRIPE_COLOR" value="be9117" />
+ <option name="myName" value="Non-TeamCity Warning" />
+ <option name="myVal" value="50" />
+ <option name="myExternalName" value="Non-TeamCity Warning" />
+ <option name="myDefaultAttributes">
+ <option name="ERROR_STRIPE_COLOR" value="be9117" />
+ </option>
+ </info>
+ <list size="8">
+ <item index="0" class="java.lang.String" itemvalue="INFORMATION" />
+ <item index="1" class="java.lang.String" itemvalue="Non-TeamCity Warning" />
+ <item index="2" class="java.lang.String" itemvalue="TYPO" />
+ <item index="3" class="java.lang.String" itemvalue="SERVER PROBLEM" />
+ <item index="4" class="java.lang.String" itemvalue="WEAK WARNING" />
+ <item index="5" class="java.lang.String" itemvalue="INFO" />
+ <item index="6" class="java.lang.String" itemvalue="WARNING" />
+ <item index="7" class="java.lang.String" itemvalue="ERROR" />
+ </list>
</settings>
</component>
\ No newline at end of file
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org