Skip to content

[build tools] Use maven batch-mode#70

Merged
wing328 merged 1 commit intoOpenAPITools:masterfrom
bmordue:maven-batch
May 16, 2018
Merged

[build tools] Use maven batch-mode#70
wing328 merged 1 commit intoOpenAPITools:masterfrom
bmordue:maven-batch

Conversation

@bmordue
Copy link
Contributor

@bmordue bmordue commented May 16, 2018

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 and ./bin/security/{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\.
  • Filed the PR against the correct branch: Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

Updated scripts in bin/ to run maven in batch mode, ie mvn -B. This means that download progress is not shown in the console. Maven's progress logging inflates the log size for CI builds and makes it harder to read through those logs.

@wing328 wing328 added this to the 3.0.0 milestone May 16, 2018
@wing328 wing328 merged commit 0d82971 into OpenAPITools:master May 16, 2018
@wing328
Copy link
Member

wing328 commented May 16, 2018

@bmordue bmordue deleted the maven-batch branch May 16, 2018 15:00
@bmordue bmordue mentioned this pull request May 21, 2018
4 tasks
wing328 pushed a commit that referenced this pull request Oct 7, 2022
Signed-off-by: tim.quinn@oracle.com <tim.quinn@oracle.com>

Signed-off-by: tim.quinn@oracle.com <tim.quinn@oracle.com>
wing328 pushed a commit that referenced this pull request Oct 16, 2022
Signed-off-by: tim.quinn@oracle.com <tim.quinn@oracle.com>

Signed-off-by: tim.quinn@oracle.com <tim.quinn@oracle.com>
nilskuhn pushed a commit to nilskuhn/openapi-generator that referenced this pull request Apr 6, 2023
…c-release-monorepo

chore(deps): update dependency semantic-release to v17.2.0
Sign up for free to 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.

2 participants