Skip to content

Minor improvement to asciidoc doc generator - #3889

Merged
wing328 merged 2 commits into
masterfrom
asciidoc_enhance
Sep 14, 2019
Merged

Minor improvement to asciidoc doc generator#3889
wing328 merged 2 commits into
masterfrom
asciidoc_enhance

Conversation

@wing328

@wing328wing328 commented Sep 14, 2019

Copy link
Copy Markdown
Member

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
  • Filed the PR against the correct branch: master, 4.1.x, 5.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

  • add copyright
  • format code
  • change LOGGER.info to LOGGER.debug
  • change permission on shell script

cc @man-at-home

public void processOpenAPI(OpenAPI openAPI) {
if (this.includeSpecMarkupLambda != null) {
LOGGER.info("specs: " + ": " + this.includeSpecMarkupLambda.resetCounter());
LOGGER.debug("specs: " + ": " + this.includeSpecMarkupLambda.resetCounter());

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

@man-at-home I changed these to LOGGER.debug. Hope that's ok with you.

@wing328
wing328 merged commit e0b5650 into masterSep 14, 2019
@wing328
wing328 deleted the asciidoc_enhance branch September 14, 2019 12:31
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@wing328