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

Trigger docs build on processor changes #42076

Merged
merged 1 commit into from
Jul 23, 2024
Merged

Conversation

radcortez
Copy link
Member

@radcortez radcortez commented Jul 23, 2024

#41979 caused an NPE in certain cases processing the configuration documentation, which the CI did not catch because the main PR build skips the docs, and the separate doc workflow does not trigger on changes to the annotation processor.

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Jul 23, 2024
Copy link

quarkus-bot bot commented Jul 23, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 0c959b7.

Failing Jobs

⚠️ Artifacts of the workflow run were not available thus the report misses some details.

Status Name Step Failures Logs Raw logs Build scan
Documentation Build Build ⚠️ Check → Logs Raw logs 🚧

@gsmet gsmet merged commit a4e611a into quarkusio:main Jul 23, 2024
3 of 4 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.14 - main milestone Jul 23, 2024
Copy link

🙈 The PR is closed and the preview is expired.

gsmet added a commit to gsmet/quarkus that referenced this pull request Aug 13, 2024
The annotation processor was only added in for the push in quarkusio#42076 and we
need to run it also for the PR.
In passing, fixed some other stuff.
danielsoro pushed a commit to danielsoro/quarkus that referenced this pull request Sep 20, 2024
The annotation processor was only added in for the push in quarkusio#42076 and we
need to run it also for the PR.
In passing, fixed some other stuff.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants