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

Add workflow to generate API client code in tidalapi module #133

Merged

Conversation

michpohl
Copy link
Contributor

@michpohl michpohl commented Oct 29, 2024

This PR adds the workflow to re-generate the API code in the :tidalapi module.

The workflow is (for now) manually triggered. A pull request created by it looks like this.

This workflow:

  1. downloads the latest available release from our openapi fork
  2. Runs the generator script to download and merge the schemas and feed them to the generator
  3. If it finds changes creates a PR with them

@michpohl michpohl force-pushed the michael/Add-workflow-to-re-generate-TidalApi-module-code branch 2 times, most recently from dfa0cc9 to ad34e63 Compare October 29, 2024 13:15
@michpohl michpohl marked this pull request as ready for review October 29, 2024 13:21
@michpohl michpohl requested a review from a team as a code owner October 29, 2024 13:21
@michpohl michpohl requested a review from e-kononenko October 29, 2024 13:21
e-kononenko
e-kononenko previously approved these changes Oct 29, 2024
Copy link

@e-kononenko e-kononenko left a comment

Choose a reason for hiding this comment

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

Very cool!!! 😎

I will borrow some ideas to iOS later if you don't mind 🙂

tidalapi/bin/generate-api-files.py Show resolved Hide resolved
.github/workflows/regenerate_tidalapi_module.yml Outdated Show resolved Hide resolved
.github/workflows/regenerate_tidalapi_module.yml Outdated Show resolved Hide resolved
This workflow:
1. downloads the latest available release from our openapi fork
2. Runs the generator script to download and merge the schemas and feed them to the generator
3. If it finds changes creates a PR with them
If running the script locally, you might either manually place the jar there, or change this entry to point to the location of your choice.
The tidal-kotlin one was only for development and doesn't exist in the fork anymore
@michpohl michpohl force-pushed the michael/Add-workflow-to-re-generate-TidalApi-module-code branch from ad34e63 to 6c0986f Compare October 29, 2024 13:57
@michpohl
Copy link
Contributor Author

Very cool!!! 😎

I will borrow some ideas to iOS later if you don't mind 🙂

Please do!

@michpohl michpohl enabled auto-merge October 29, 2024 14:23
e-kononenko
e-kononenko previously approved these changes Oct 29, 2024
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3
Copy link
Contributor

Choose a reason for hiding this comment

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

The other workflows in tidal-sdk-android are already using checkout@v4. It shouldn't be a problem, just fyi for consistency.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no you are right. This should be updated. Thanks!

@michpohl michpohl added this pull request to the merge queue Oct 30, 2024
Merged via the queue into main with commit b467d8d Oct 30, 2024
7 checks passed
@michpohl michpohl deleted the michael/Add-workflow-to-re-generate-TidalApi-module-code branch October 30, 2024 10:53
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 this pull request may close these issues.

3 participants