-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
project-keeper/src/main/java/com/exasol/projectkeeper/mavenrepo/MavenRepository.java
Show resolved
Hide resolved
...ect-keeper/src/main/java/com/exasol/projectkeeper/validators/files/FileTemplatesFactory.java
Show resolved
Hide resolved
on: | ||
schedule: | ||
- cron: "0 5 * * 0" | ||
push: | ||
branches: | ||
- main | ||
pull_request: |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
project-keeper/src/main/resources/templates/.github/workflows/ci-build-next-java.yml
Outdated
Show resolved
Hide resolved
on: | ||
schedule: | ||
- cron: "0 5 * * 0" | ||
push: | ||
branches: | ||
- main | ||
pull_request: |
There was a problem hiding this comment.
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?
on: | ||
schedule: | ||
- cron: "0 5 * * 0" | ||
push: | ||
branches: | ||
- main | ||
pull_request: |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
11 | ||
17 |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
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
.
...keeper/src/test/java/com/exasol/projectkeeper/validators/files/JavaVersionExtractorTest.java
Outdated
Show resolved
Hide resolved
…rs/files/JavaVersionExtractorTest.java Co-authored-by: Christoph Kuhnke <[email protected]>
Quality Gate passedIssues Measures |
Closes #597