Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fail spec creation if asciidoctor errors are encountered #1205

Merged
merged 2 commits into from
Jul 10, 2024

Conversation

kpet
Copy link
Contributor

@kpet kpet commented Jul 10, 2024

This would have enabled the CI to catch a markup issue introduced by #938.

Change-Id: I49de3eaf623117f7c29d1019dedf5b342766a029

@kpet
Copy link
Contributor Author

kpet commented Jul 10, 2024

The CI is failing, as expected, because the spec source in main triggers asciidoctor errors. We will need at least #1206 to make the CI pass.

@bashbaug bashbaug requested a review from oddhack July 10, 2024 15:21
@bashbaug
Copy link
Contributor

I just merged #1206. Could you merge/rebase to pick up the changes and we should see CI passing again?

I'd also be interested in comments from @oddhack - thanks!

This would have enabled the CI to catch a markup issue introduced by KhronosGroup#938.

Signed-off-by: Kevin Petit <kevin.petit@arm.com>
Change-Id: I49de3eaf623117f7c29d1019dedf5b342766a029
@kpet kpet force-pushed the fail-ci-on-asciidoctor-error branch from ae6d0b0 to d827bba Compare July 10, 2024 15:26
@kpet
Copy link
Contributor Author

kpet commented Jul 10, 2024

It's still failing after a rebase. I'm looking into the failures.

Change-Id: I0f9cbeddb72e0d76ba508b336d91c4ee640d77ad
@kpet
Copy link
Contributor Author

kpet commented Jul 10, 2024

Now passing with fixes to a few tables. The errors weren't flagged by my local version of asciidoctor.

Copy link
Contributor

@oddhack oddhack left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

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

Thanks!

@bashbaug bashbaug merged commit 735396c into KhronosGroup:main Jul 10, 2024
2 checks passed
@kpet kpet deleted the fail-ci-on-asciidoctor-error branch July 13, 2024 09:38
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