Skip to content

Issues #1942, #1943, #1944, #1952#6784

Merged
hansva merged 10 commits intoapache:mainfrom
mattcasters:issue-1942
Mar 18, 2026
Merged

Issues #1942, #1943, #1944, #1952#6784
hansva merged 10 commits intoapache:mainfrom
mattcasters:issue-1942

Conversation

@mattcasters
Copy link
Copy Markdown
Contributor

@mattcasters mattcasters commented Mar 14, 2026

#1942 : CSV Input
#1943 : Text File Input
#1944 : Text File Output
#1952 : JSON Input/Output/Enhanced

For every touched class...

  • Implemented Lombok
  • Fixed typos and spelling errors
  • Fixed a lot of warnings
  • I added a round trip XML tester
  • I minimized any changes to the non-Meta classes, leaving functionality alone.
  • Fixed metadata injection
  • Added extra integration tests for metadata injection in case none were present

@mattcasters mattcasters marked this pull request as ready for review March 18, 2026 14:06
@hansva hansva merged commit 6bdc5fd into apache:main Mar 18, 2026
3 checks passed
jefersonnavarro pushed a commit to jefersonnavarro/hop that referenced this pull request Mar 24, 2026
* Issues apache#1942, apache#1943, apache#1944, apache#1952

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (further tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (test hardening)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (test hardening)
jefersonnavarro pushed a commit to jefersonnavarro/hop that referenced this pull request Mar 24, 2026
* Issues apache#1942, apache#1943, apache#1944, apache#1952

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (further tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (more fixes, tests and improvements)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (test hardening)

* Issues apache#1942, apache#1943, apache#1944, apache#1952 (test hardening)
@mattcasters mattcasters deleted the issue-1942 branch April 1, 2026 17:55
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.

2 participants