Skip to content

Conversation

etherealjoy
Copy link
Contributor

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: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

linux is a reserved keyword for compilation with gcc in linux for C++.

Test,

  • PetStore not affected
  • Docker API can be generated without API modifications with same functionality

@stkrwork @fvarose @ravinikam

@etherealjoy etherealjoy changed the title Cpp linux reserve keyword [C++] Add linux as a reserve keyword Mar 15, 2018
@wing328 wing328 added this to the v2.4.0 milestone Mar 15, 2018
@wing328 wing328 merged commit e796e4c into swagger-api:master Mar 15, 2018
@etherealjoy etherealjoy deleted the cpp_linux_reserve_keyword branch March 15, 2018 14:00
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.

3 participants