-
Notifications
You must be signed in to change notification settings - Fork 21
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
Actions pipeline fails with official pip install in case of incompatible changes #18
Comments
@tcitbiint3 I don't quite understand, what does it mean "incompatible changes"? |
By the way, could you change the string to |
Yes, changes which influence the output of the example or changes which change a function header in a incompatible way, e.g. when defining a parameter mandatory which was optional before, so that the execution of the examples is not possible with the version behind pip install. Using the current version of the repository will test the current version, which makes more sense or am I wrong? In case of PR #17, in which the example was extended, and the current "python -m pip install code-generation", a new version has to be pushed to pip in order to get Actions pipeline without errors. |
"which change a function header in a incompatible way, e.g. when defining a parameter mandatory which was optional before" yeah, I met such inconveniences before. |
BTW, we have an assertion after the merge |
Yes, this is because of the test execution with v2.1.1 instead of the version of the PR. in the PR, the tests are changed in order to test the changes properly. Therefore |
Okay, let's try out |
I fixed it with my package release script. |
This looks like the best solution! Thanks! |
@tcitbiint3 I want to move the library to another level and include support for other languages. |
@yuchdev
Using the pip version of the project will lead to a failing Actions pipeline in case of incompatible changes:
code_generator/.github/workflows/python-app.yml
Line 36 in 304b7b6
Potential bugfix available with PR #17
Is this the intended behavior?
The text was updated successfully, but these errors were encountered: