Skip to content

feat(spec): Add $self for identity-based referencing#450

Closed
frankkilcommins wants to merge 1 commit intoOAI:v1.1-devfrom
frankkilcommins:self-feature
Closed

feat(spec): Add $self for identity-based referencing#450
frankkilcommins wants to merge 1 commit intoOAI:v1.1-devfrom
frankkilcommins:self-feature

Conversation

@frankkilcommins
Copy link
Copy Markdown
Collaborator

fixes: #434

Took the opportunity to restructure some section locations too for improved readability.

Copy link
Copy Markdown
Member

@handrews handrews left a comment

Choose a reason for hiding this comment

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

Doing the re-org at the same time as adding a new feature makes this very hard to review. Both of these things are potentially complex (either conceptually or mechanically). Would it be possible to split this into two PRs?

@@ -0,0 +1,14 @@
arazzo: 1.1.0
$self: https://example.com/workflows/minimal-with-self.arazzo.yaml#/workflows/basicWorkflow
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No fragment 🙂

@frankkilcommins
Copy link
Copy Markdown
Collaborator Author

closing this in favour of #451 which just has changes for $self and added clarity on document parsing. The broader structural changes will happen in a separate PR and are unrelated to this specific feature.

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.

2 participants