-
Notifications
You must be signed in to change notification settings - Fork 401
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
Remove support for var-args in plc4j #1739
Conversation
acf938a
to
27f74fa
Compare
Checked it out, built it locally and ran the manual tests against my equipment. All worked fine. Also had a look at the changes in the drivers and liked what I saw. |
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.
LGTM
Great, will rebase and push changes. |
I think if you rebase a branch that you already pushed, that's when you get these nasty github warnings ... I only rebase "develop" and merge all others ... but please ... don't merge before the build passes ;-) |
27f74fa
to
cdc17a4
Compare
You push changes faster than CI completes jobs. :-) |
…cks of generated contents. BREAKING CHANGE: Removed `staticParse(io, ... args)` methods. Signed-off-by: Łukasz Dywicki <[email protected]>
Signed-off-by: Łukasz Dywicki <[email protected]>
cdc17a4
to
aaf78d9
Compare
No description provided.