-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Storage queue from Alan's PR #4036
Changes from 136 commits
5082438
6bfa0b0
a991f24
13eec1b
f59d502
ab33289
3db312b
80153a2
710d8cc
d265fdd
fe57bfa
d9b6bbc
75a28cd
300dadc
3b28219
22e32f5
d1ce6e8
764e6cb
044b3b2
64625d7
f3c8f06
1e46e4b
ebb0d81
1181a67
fcc7308
5fb171d
83e1846
5315982
3248745
1fc2fa1
472102c
c6540a9
15c7bb7
fe46421
74ed144
1bb1e5e
aaa1c7d
c2d1c7c
e1439ff
4fab228
e93c63b
3e8e568
bf6dad2
9ff7c28
40e177d
50faad9
955672f
0154afc
90dfb90
be2ac22
7b4d671
6f6d336
3afae03
9856fc4
248a409
9a57ab3
22c29ed
bbfdb44
7e42e16
ec95c0a
7e6fd14
a8be98f
02e6334
7fd418f
af68457
0b83213
c4a9106
064da3a
cadbf79
626c373
f538bcd
287557c
d8cffe5
4e64a50
56cf6f5
4d52b40
607708e
e9bce28
91c8651
106786a
c381d26
319c1b4
3e2b62e
0515152
70a8fa8
2312e42
ea70315
9d7e811
77419e5
61bdca8
f12c903
6ede17a
6050497
04d31b8
70d303e
28860df
f170c05
4d80982
8f17c71
42d354f
d967764
38df5f7
eee7de1
441410f
7490012
0a28789
f047f50
a8228da
e0df8f6
0435b63
a3a3c81
8a8d242
d9639cb
00fcd74
8fc5051
f06b70f
7c7f043
ccd014c
a767579
f114f4e
6137b5a
c8b524e
5cfbf64
f2bff76
d32e978
b99c3ef
7e328f1
6c2fc73
73e8ed3
124173f
d4cb144
674e041
2935bd3
a235a9b
dd7e986
e6342aa
a48efde
9f1b5b0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -89,8 +89,7 @@ | |
|
||
<build> | ||
<plugins> | ||
<!-- Now that all checkstyle issues have been resolved, override the | ||
default behavior to make checkstyle fail on error and violation. --> | ||
<!-- For initial implementation all Checkstyle, SpotBugs, and JavaDocs errors will be ignored --> | ||
<plugin> | ||
<groupId>org.apache.maven.plugins</groupId> | ||
<artifactId>maven-checkstyle-plugin</artifactId> | ||
|
@@ -115,6 +114,23 @@ | |
<failOnError>false</failOnError> | ||
</configuration> | ||
</plugin> | ||
|
||
<plugin> | ||
<groupId>org.apache.maven.plugins</groupId> | ||
<artifactId>maven-javadoc-plugin</artifactId> | ||
<configuration> | ||
<failOnError>true</failOnError> | ||
<failOnWarnings>true</failOnWarnings> | ||
</configuration> | ||
</plugin> | ||
|
||
<plugin> | ||
<groupId>com.github.spotbugs</groupId> | ||
<artifactId>spotbugs-maven-plugin</artifactId> | ||
<configuration> | ||
<failOnError>false</failOnError> | ||
</configuration> | ||
</plugin> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These changes can all be removed - it is centrally managed now. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Checkstyle should be removed too, right? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, all configuration of these features is centralised |
||
</plugins> | ||
</build> | ||
</project> |
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.
There shouldn't be
javadoc
at the end of this line