You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2021/02/08 20:54:13 UTC
[GitHub] [lucene-solr] epugh opened a new pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
epugh opened a new pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325
# Description
Missing `package-info.java` causes a NPE in the javadocs task and is not obvious what went wrong!
# Solution
Look for the very specific situation, and then don't include the `Element` in the `reporter.print()` method. Maybe this should be a more generic test, but I wanted to be careful of not breaking other things.
# Tests
Delete a `package-info.java`, and then run javadocs and you'll see the error!
# Checklist
Please review the following and check all that apply:
- [X ] I have reviewed the guidelines for [How to Contribute](https://wiki.apache.org/solr/HowToContribute) and my code conforms to the standards described there to the best of my ability.
- [X ] I have created a Jira issue and added the issue ID to my pull request title.
- [ X] I have given Solr maintainers [access](https://help.github.com/en/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork) to contribute to my PR branch. (optional but recommended)
- [ X] I have developed this patch against the `master` branch.
- [ X] I have run `./gradlew check`.
- [ ] I have added tests for my changes.
- [ ] I have added documentation for the [Ref Guide](https://github.com/apache/lucene-solr/tree/master/solr/solr-ref-guide) (for Solr changes only).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] dweiss commented on pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
dweiss commented on pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325#issuecomment-775462215
Strange. Can you rebase on top of master?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] muse-dev[bot] commented on a change in pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
muse-dev[bot] commented on a change in pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325#discussion_r572425552
##########
File path: dev-tools/missing-doclet/src/main/java/org/apache/lucene/missingdoclet/MissingDoclet.java
##########
@@ -430,6 +430,11 @@ private void error(Element element, String message) {
fullMessage.append(element.getKind().toString().toLowerCase(Locale.ROOT));
fullMessage.append("): ");
fullMessage.append(message);
- reporter.print(Diagnostic.Kind.ERROR, element, fullMessage.toString());
+ if (((element.getKind() == ElementKind.PACKAGE) && (fullMessage.toString().contains("javadocs are missing")))){
Review comment:
*UnnecessaryParentheses:* Unnecessary use of grouping parentheses
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] epugh commented on pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
epugh commented on pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325#issuecomment-775461334
I often get this when I branch.. Let me try again.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] dweiss commented on pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
dweiss commented on pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325#issuecomment-775457028
Hmmm... something went wrong in this pull request? Why so many commits?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] epugh commented on pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
epugh commented on pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325#issuecomment-775482479
@dweiss I ended up making a new PR: https://github.com/apache/lucene-solr/pull/2326
Thanks for looking at this!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
[GitHub] [lucene-solr] epugh closed pull request #2325: LUCENE-9747: Missing package-info.java causes NPE in MissingDoclet.java
Posted by GitBox <gi...@apache.org>.
epugh closed pull request #2325:
URL: https://github.com/apache/lucene-solr/pull/2325
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org