Module layout and use of Step classes on new contributions #129
dannymeijer
started this conversation in
Ideas
Replies: 2 comments
-
add #131 to the list of the PRs I mentioned prior. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I opened #141 to help make life easier. Once this gets merged, you can use the pandas based Transformation class to restructure your code. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Recently, we've been getting quite a few PRs opened to extend Koheesio with various Transformation style functions.
Unfortunately, all of these PRs (#118, #119, #122, #123, #127, and #128 so far) are not following the intended code style of Koheesio.
Since these PRs are very similar in nature, I want to use this discussion to come up with a proper strategy on how to incorporate these changes properly.
What I would like to see for these contributions is:
Please let me know if and how I can help. Would love to do a zoom call with you to explain or even pair program.
Beta Was this translation helpful? Give feedback.
All reactions