Skip to content

[Fix #1772] Document jackson dependecy. - #1779

Open
fjtirado wants to merge 2 commits into
open-workflow-specification:mainfrom
fjtirado:Fix_#1772
Open

fjtirado wants to merge 2 commits into
open-workflow-specification:mainfrom
fjtirado:Fix_#1772

Conversation

@fjtirado

Copy link
Copy Markdown
Collaborator

Fix #1772

Copilot AI balanced review requested due to automatic review settings October 20, 2026 17:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The documentation-only change has no blocking issues; only minor wording corrections remain.

No open issues 🎉

What changed in this PR

Documents Jackson dependencies and the existing alternative for users of other JSON libraries, providing context for #1772 without changing module structure.

Changes:

  • Explains Jackson’s role in openworkflow-api.
  • Describes using openworkflow-types to build alternative readers and writers.
File Description
README.md Adds guidance on Jackson dependencies and library-independent POJOs.

Review effort: Balanced

Signed-off-by: Francisco Javier Tirado Sarti <ftirados@ibm.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 18:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No blocking issues remain; the retained feedback is an optional documentation correction.

1 open issue

Review effort: Balanced

Comment thread README.md Outdated
Copilot AI balanced review requested due to automatic review settings October 20, 2026 18:19
Clarified the Jackson dependency information and improved readability.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Francisco Javier Tirado Sarti <ftirados@ibm.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The documentation reflects the existing module structure, and the exception cleanup is consistent with Jackson 3’s unchecked exceptions.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

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.

Separate Jackson specific workflow from openworkflow-api module

3 participants