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

Major upgrade Python Generator #156

Open
victorskl opened this issue Jul 31, 2024 · 2 comments · Fixed by #160
Open

Major upgrade Python Generator #156

victorskl opened this issue Jul 31, 2024 · 2 comments · Fixed by #160
Assignees
Labels
Milestone

Comments

@victorskl
Copy link
Member

Context:

In libica-2.x,

We use 5.4.0 (LTS) throughout for ICAv2:

https://github.com/umccr-illumina/libica/blob/a735bb7/.openapi-generator/VERSION

And, 4.3.0 (LTS) throughout for ICAv1:

https://github.com/umccr-illumina/libica/blob/c320a6d/.openapi-generator/VERSION

Upgrade:

We shall upgrade this to latest generator 7.x when possible.

https://github.com/OpenAPITools/openapi-generator?tab=readme-ov-file#11---compatibility

Caveats:

This may likely break backward compatibility to sudden extent. It also depends on a generator choice.

@victorskl victorskl added the epic label Jul 31, 2024
@victorskl victorskl added this to the 3.0 milestone Jul 31, 2024
@victorskl
Copy link
Member Author

Also.

Explore alternative generators as replacement for more idiomatic, Pythonic, modern Python semantic.

Caveat is high risk with generated SDK code stability due to low adoption & user base.

Candidates:

@victorskl
Copy link
Member Author

I am starting v3 package soon.

In v3 roadmap;

  • I will retain current v2 as LTS purpose. (Just like v1 (LTS) <-> v2 transition)
  • And, remove v1 packages completely.
  • And, add deprecation marker to app package to redirect wrapica.

@victorskl victorskl linked a pull request Aug 24, 2024 that will close this issue
@victorskl victorskl self-assigned this Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant