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

fix #2806 #2837

Merged
merged 2 commits into from
Apr 24, 2023
Merged

fix #2806 #2837

merged 2 commits into from
Apr 24, 2023

Conversation

sramazzina
Copy link
Contributor

@sramazzina sramazzina commented Apr 20, 2023

fix #2806 - Closing this issue by better specifying the transform behavior in documentation. We will postpone the task of refactoring the way trim is applied all around in Hop platform. See #2836


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@hansva hansva merged commit c829907 into apache:master Apr 24, 2023
@sramazzina sramazzina deleted the 2806 branch April 27, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: The "Database Lookup" step trim function does not seem to work
2 participants