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
GH-321: [Java] add new error expiration as part of endpoints information #322
GH-321: [Java] add new error expiration as part of endpoints information #322
Conversation
Also adding configuration of |
To fix error: https://github.com/apache/arrow-cookbook/actions/runs/5972085567/job/16202106444?pr=322
|
java/ext/javadoctest.py
Outdated
@@ -105,6 +105,7 @@ def setup(app) -> Dict[str, Any]: | |||
app.add_directive("testoutput", TestoutputDirective) | |||
app.add_builder(JavaDocTestBuilder) | |||
# this config value adds to sys.path |
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.
The comment now is with the wrong line, can you move it?
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.
The comment now is with the wrong line, can you move it?
Please note that all configurations below are subject to this comment.
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.
No: the comment is talking about doctest_path
specifically.
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.
I will update them now
java/ext/javadoctest.py
Outdated
@@ -105,6 +105,7 @@ def setup(app) -> Dict[str, Any]: | |||
app.add_directive("testoutput", TestoutputDirective) | |||
app.add_builder(JavaDocTestBuilder) | |||
# this config value adds to sys.path |
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.
No: the comment is talking about doctest_path
specifically.
To close #321