Skip to content

chore: prepare jsonpath pipeline#621

Merged
zregvart merged 1 commit intoapache:mainfrom
zregvart:pr/configure-jsonpath-extension
Sep 9, 2021
Merged

chore: prepare jsonpath pipeline#621
zregvart merged 1 commit intoapache:mainfrom
zregvart:pr/configure-jsonpath-extension

Conversation

@zregvart
Copy link
Member

@zregvart zregvart commented Sep 9, 2021

This is the last bit needed for the changes in apache/camel#6040

To get merged, utilizing the jsonpath macro in the Antora pipeline to generate component/endpoint options table.

This has been picked from #617 to ease merging. The bit that was not included in this or in the PR #620 is the refactoring around absolute/relative URLs, i.e. the patch-sitemap.js is still present after this.

@zregvart zregvart requested a review from djencks September 9, 2021 08:19
@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2021

🚀 Preview is available at https://pr-621--camel.netlify.app

This is the last bit needed for the changes in

apache/camel#6040

To get merged, utilizing the jsonpath macro in the Antora pipeline to
generate component/endpoint options table.

This has been picked from

apache#617

to ease merging. The bit that was not included in this or in the PR apache#620
is the refactoring around absolute/relative URLs, i.e. the
`patch-sitemap.js` is still present after this.
@zregvart zregvart force-pushed the pr/configure-jsonpath-extension branch from 67048ed to 896132c Compare September 9, 2021 09:36
@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2021

🚀 Preview is available at https://pr-621--camel.netlify.app

Copy link
Contributor

@djencks djencks left a comment

Choose a reason for hiding this comment

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

LGTM

@zregvart zregvart merged commit 479c07c into apache:main Sep 9, 2021
@zregvart zregvart deleted the pr/configure-jsonpath-extension branch September 9, 2021 18:01
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.

3 participants