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

#597: Move workflows into ci-build.yml #605

Merged
merged 14 commits into from
Nov 22, 2024
Merged

Conversation

kaklakariada
Copy link
Collaborator

Closes #597

@kaklakariada kaklakariada enabled auto-merge (squash) November 21, 2024 15:50
Comment on lines 5 to -11
on:
schedule:
- cron: "0 5 * * 0"
push:
branches:
- main
pull_request:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Link checker workflow still required for cron job. Does not run for PR or branch changes any more.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can I still change the documentation of any project and verify if all links are valid before merging to main?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah - I think ci-build-db-version-matrix.yml is the answer.

Comment on lines 5 to -11
on:
schedule:
- cron: "0 5 * * 0"
push:
branches:
- main
pull_request:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can I still change the documentation of any project and verify if all links are valid before merging to main?

Comment on lines 5 to -11
on:
schedule:
- cron: "0 5 * * 0"
push:
branches:
- main
pull_request:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah - I think ci-build-db-version-matrix.yml is the answer.

@@ -110,6 +110,78 @@ jobs:
path: ${{ steps.build-pk-verify.outputs.release-artifacts }}
retention-days: 5

- name: Configure link check
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a way of avoiding the duplicate configuration of github-action-markdown-link-check?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, but I didn't want to do this in this PR.

Comment on lines +182 to +183
11
17
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't these versions be defined by the entries in .project-keeper.yml?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These versions are overwritten by PK based on property java.version in the pom.xml.

…rs/files/JavaVersionExtractorTest.java

Co-authored-by: Christoph Kuhnke <[email protected]>
@kaklakariada kaklakariada merged commit 8b88d4b into main Nov 22, 2024
8 checks passed
@kaklakariada kaklakariada deleted the kaklakariada/issue597 branch November 22, 2024 14:54
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.

Integrate PR jobs into ci-build.yml
2 participants