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

Setup SDK generator #79

Closed
wants to merge 8 commits into from
Closed

Conversation

andrii-balitskyi
Copy link
Collaborator

@andrii-balitskyi andrii-balitskyi commented Nov 8, 2023

Closes #78

  • generate SDK
  • fix tests
  • add renovate and seamapi/types

@andrii-balitskyi andrii-balitskyi marked this pull request as ready for review November 14, 2023 13:53
Copy link
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

Looks good, can we document in the README how to install the non-generated legacy version. This way we can tell customers to revert to it if there's an issue (since we dont have semantic versioning with php).

For the workflow we might want to just remove the conditional check and see if it causes a loop, maybe it won't 🤷

I noticed SeamError was removed, is that problematic? Maybe it just got moved? Worried about error handling for existing code

echo "Generating PHP SDK..."
else
echo "No 'autorelease' keyword found. Skipping PHP SDK generation."
exit 0
Copy link
Contributor

Choose a reason for hiding this comment

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

This wont stop the workflow i think, it'll continue to the next step (I think? Exit 0 means exit with success) you also cant error out, so there must be a different way

@andrii-balitskyi
Copy link
Collaborator Author

Was merged into https://github.com/seamapi/generated-php

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.

Autogenerated PHP SDK
2 participants