fix: Use Pydantic exclude_none=True to omit null namespace fields from catalog serialization #3586
This file contains hidden or 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: "Verify Semantic PR Title" | |
| on: | |
| pull_request: | |
| types: | |
| - opened | |
| - edited | |
| - synchronize | |
| - ready_for_review | |
| env: | |
| AIRBYTE_ANALYTICS_ID: ${{ vars.AIRBYTE_ANALYTICS_ID }} | |
| permissions: | |
| pull-requests: read | |
| jobs: | |
| validate_pr_title: | |
| name: Validate PR title | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: amannn/action-semantic-pull-request@48f256284bd46cdaab1048c3721360e808335d50 # v6.1.1 | |
| if: ${{ github.event.pull_request.draft == false }} | |
| env: | |
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |
| with: | |
| # Configure which types are allowed (newline-delimited). | |
| # These are intentionally case-insensitive, allowing title casing or all lowercase. | |
| # See: https://github.com/commitizen/conventional-commit-types/blob/master/index.json | |
| types: | | |
| fix | |
| Fix | |
| feat | |
| Feat | |
| docs | |
| Docs | |
| ci | |
| CI | |
| chore | |
| Chore | |
| build | |
| Build | |
| test | |
| Test | |
| # # We don't use scopes as of now | |
| # scopes: | | |
| # core | |
| # ui | |
| - name: Check for "do not merge" in PR title | |
| if: ${{ github.event.pull_request.draft == false }} | |
| uses: actions/github-script@f28e40c7f34bde8b3046d885e986cb6290c5673b # v7.1.0 | |
| with: | |
| script: | | |
| const title = context.payload.pull_request.title.toLowerCase(); | |
| if (title.includes('do not merge') || title.includes('do-not-merge')) { | |
| core.setFailed('PR title contains "do not merge" or "do-not-merge". Please remove this before merging.'); | |
| } |