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

Feature/plc4py #343

Merged
merged 46 commits into from Apr 9, 2022
Merged

Feature/plc4py #343

merged 46 commits into from Apr 9, 2022

Conversation

hutcheb
Copy link
Contributor

@hutcheb hutcheb commented Apr 9, 2022

Merge plc4py into develop

ottlukas and others added 30 commits November 10, 2019 12:19
Feature/plc4py
Initial setup and poetry project
black, mypy and pre-commit. pre-commit is added as a dev dependency in poetry

To enable these hooks ->

- pip install pre-commit
- pre-commit install

Please see [pre-commit](https://pre-commit.com/#usage)

A local .gitignore specific to python was also added
PLC4X-236 Pre-Commit and Hinting Configuration for python
PLC4X-250 python Namespace refactor - for python namespace this makes sense so we will go with that
@hutcheb
Copy link
Contributor Author

hutcheb commented Apr 9, 2022

Noting the Plc4Py Interop binding will be remove with this merge.

If anyone has a concern we can put it back in if need be.

@hutcheb hutcheb merged commit f819208 into develop Apr 9, 2022
@hutcheb hutcheb deleted the feature/plc4py branch April 9, 2022 21:24
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.

None yet

6 participants