Skip to content
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

Elixir - generated code has errors and warnings on latest (1.7) elixir (pr incoming) #12465

Open
fcbry opened this issue Sep 13, 2024 · 0 comments · May be fixed by #12466
Open

Elixir - generated code has errors and warnings on latest (1.7) elixir (pr incoming) #12465

fcbry opened this issue Sep 13, 2024 · 0 comments · May be fixed by #12466

Comments

@fcbry
Copy link

fcbry commented Sep 13, 2024

Description

Hi @niku - thanks for your library - it's been a great help to me.

When testing it against teamwork.com swagger definition (my fork with fixes to make it compatible with swagger-codegen (changed nil to 0)) the generated code had a couple of (thousand) quoted atom warnings and some api binding errors, with the changes in the incomming PR those errors and warnings will be fixed.

I have also cherry picked my as unmerged PR commit #12464 as without it, the mvn package task fails to package modules/swagger-codegen-cli/target/swagger-codegen-cli.jar - lets see which gets merged first.

Swagger-codegen version

master (61cfeac)

Swagger declaration file content or url

https://gist.github.com/fcbry/465e767f059504c05717fc715345bf22

Command line used for generation
mvn package -DskipTests  -D"maven.test.skip=true" && java -jar modules/swagger-codegen-cli/target/swagger-codegen-cli.jar generate \
    -i "C:\Users\BryanHunt\Downloads\teamwork.com-api-object-reference.oas.yml" \
    -l elixir -D"packageName=fc_teamwork" -D"invokerPackage=FC.Teamwork" \
    -o C:\code\fcbry_teamwork_poc\apps\fc_teamwork_client
Steps to reproduce

As above

Related issues/PRs

No

Suggest a fix/enhancement

I'm providing a PR to fix this

@fcbry fcbry linked a pull request Sep 13, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant