-
Notifications
You must be signed in to change notification settings - Fork 4
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
Conversation
… setup-for-sdk-generator
There was a problem hiding this 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 |
There was a problem hiding this comment.
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
Was merged into https://github.com/seamapi/generated-php |
Closes #78