Skip to content

Conversation

@mwiebe
Copy link
Contributor

@mwiebe mwiebe commented Jan 30, 2026

What was the problem/requirement? (What/Why)

RFC 4 is accepted and needs implementation

What was the solution? (How)

Update the openjd-model dependency to 0.9.* that supports FEATURE_BUNDLE_1.

Implement support for the endOfLine specification in embedded files in templates. The syntax sugar is de-sugared in openjd-models, and the limits are enforced there, so this is the only change that openjd-sessions needs to perform.

Where some model field types changed as noted in the openjd-model changelog, have commented # type: ignore[arg-type] to accept the types as the runtime behavior there is unchanged.

What is the impact of this change?

Support for OpenJD RFC 0004.

How was this change tested?

Tested together with the openjd-model and an openjd-cli change.

Was this change documented?

N/A

Is this a breaking change?

No

Does this change impact security?

No


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

jericht
jericht previously approved these changes Jan 30, 2026
Signed-off-by: Mark <399551+mwiebe@users.noreply.github.com>
@mwiebe mwiebe force-pushed the implement-rfc-0004 branch from 40e8a87 to 47521c8 Compare February 3, 2026 18:52
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

@mwiebe mwiebe marked this pull request as ready for review February 3, 2026 18:55
@@ -1,6 +1,7 @@
# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we build some job bundles and add them to integration testing at the openjd cli level?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, there are a few simple ones in https://github.com/OpenJobDescription/openjd-cli/pull/204/changes, we can expand that.

@mwiebe mwiebe merged commit b9f3b8c into OpenJobDescription:mainline Feb 3, 2026
25 checks passed
@mwiebe mwiebe deleted the implement-rfc-0004 branch February 3, 2026 19:27
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.

3 participants