fix: archive - simplify and enhance async retention policy application #2210
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: "Lint PR" | |
on: | |
pull_request_target: | |
types: | |
- opened | |
- edited | |
- synchronize | |
jobs: | |
main: | |
name: Validate PR title | |
runs-on: ubuntu-latest | |
permissions: | |
pull-requests: write | |
steps: | |
- uses: amannn/action-semantic-pull-request@v5 | |
id: lint_pr_title | |
with: | |
types: | | |
chore | |
docs | |
feat | |
fix | |
refactor | |
style | |
test | |
env: | |
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |
- uses: marocchino/sticky-pull-request-comment@v2 | |
# When the previous steps fails, the workflow would stop. By adding this | |
# condition you can continue the execution with the populated error message. | |
if: always() && (steps.lint_pr_title.outputs.error_message != null) | |
with: | |
header: pr-title-lint-error | |
message: | | |
Hey there and thank you for opening this pull request! 👋🏼 | |
We require pull request titles to follow the [Conventional Commits specification](https://www.conventionalcommits.org/en/v1.0.0/) and it looks like your proposed title needs to be adjusted. | |
Details: | |
> ${{ steps.lint_pr_title.outputs.error_message }} | |
# Delete a previous comment when the issue has been resolved | |
- if: ${{ steps.lint_pr_title.outputs.error_message == null }} | |
uses: marocchino/sticky-pull-request-comment@v2 | |
with: | |
header: pr-title-lint-error | |
delete: true | |
labels: | |
runs-on: ubuntu-latest | |
steps: | |
- uses: actions/checkout@v3 | |
name: Checkout code | |
id: checkout | |
- uses: dorny/paths-filter@v2 | |
id: filter | |
with: | |
filters: | | |
config: | |
- 'apps/wakunode2/external_config.nim' | |
- 'apps/networkmonitor/networkmonitor_config.nim' | |
- 'apps/chat2/config_chat2.nim' | |
- 'apps/chat2bridge/config_chat2bridge.nim' | |
db_schema: | |
- 'waku/waku_archive/driver/postgres_driver/postgres_driver.nim' | |
- 'waku/waku_archive/driver/sqlite_driver/queries.nim' | |
- name: Comment config change | |
uses: thollander/actions-comment-pull-request@v2 | |
if: ${{steps.filter.outputs.config == 'true'}} | |
with: | |
message: | | |
This PR may contain changes to **configuration options** of one of the apps. | |
If you are introducing a breaking change (i.e. the set of options in latest release would no longer be applicable) make sure the original option is preserved with a *deprecation* note for 2 following releases before it is actually removed. | |
Please also make sure the label `release-notes` is added to make sure any changes to the user interface are properly announced in changelog and release notes. | |
comment_tag: configs | |
- name: Comment DB schema change | |
uses: thollander/actions-comment-pull-request@v2 | |
if: ${{steps.filter.outputs.db_schema == 'true'}} | |
with: | |
header: pr-title-lint-error | |
message: | | |
This PR may contain changes to **database schema** of one of the drivers. | |
If you are introducing any changes to the schema, make sure the upgrade from the latest release to this change passes without any errors/issues. | |
Please make sure the label `release-notes` is added to make sure upgrade instructions properly highlight this change. | |
comment_tag: db_schema |