Skip to content

PR-DALI-1.2#67

Merged
pdowler merged 1 commit intoivoa-std:mainfrom
pdowler:main
Oct 21, 2025
Merged

PR-DALI-1.2#67
pdowler merged 1 commit intoivoa-std:mainfrom
pdowler:main

Conversation

@pdowler
Copy link
Collaborator

@pdowler pdowler commented Oct 17, 2025

target date near start of RFC: 2025-10-24
use auxiliaryurl macro for openapi component refs

target date near start of RFC: 2025-10-24
use auxiliaryurl macro for openapi component refs
Copy link
Contributor

@msdemlei msdemlei left a comment

Choose a reason for hiding this comment

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

You should add everything you auxliaryurl to to AUX_FILES in the Makefile, or they won't appear in the archive and hence be missing from the published directory.

And I'm a bit curious whether everything works with aux files in subdirectories. If ivoatex does something wonky with that, don't hesitate to complain.

@pdowler
Copy link
Collaborator Author

pdowler commented Oct 21, 2025

just mentioning the openapi subdir in AUX_FILES worked as intended; the subdir and files are included in the package and become accessible from the expected docrepo URL.

Still to be determined (in the RFC and discussion at the interop): these will end up with a versioned URL in the documents themselves and I'm not sure if/how we want to version such openapi components.... so that bit is TBD. Until we decide on that, no changes in ivoatex are needed yet.

Copy link
Contributor

@msdemlei msdemlei left a comment

Choose a reason for hiding this comment

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

Looks good, let's go.

@pdowler pdowler merged commit eae8459 into ivoa-std:main Oct 21, 2025
1 check passed
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