Message-ID: <gh-pgjdbc-pgjdbc-3257-rc1609885203@github.com>
From: "sehrope (@sehrope)" <noreply+sehrope@github.com>
To: "pgjdbc/pgjdbc" <noreply+pgjdbc-pgjdbc@github.com>
Date: Wed, 22 May 2024 12:45:23 +0000
Subject: Re: [pgjdbc/pgjdbc] PR #3257: test: add GitHub CI testing with PostgreSQL 15 and HEAD
In-Reply-To: <gh-pgjdbc-pgjdbc-3257@github.com>
References: <gh-pgjdbc-pgjdbc-3257@github.com>
List-Id: <gh-pgjdbc-pgjdbc.github.com>
X-GitHub-Author-Login: sehrope
X-GitHub-Comment-Id: 1609885203
X-GitHub-Comment-Type: review_comment
X-GitHub-Commit: 7ea63e5351e9460897165cf10e6d242042921bab
X-GitHub-Issue: 3257
X-GitHub-Line: 154
X-GitHub-Path: .github/workflows/main.yml
X-GitHub-Repo: pgjdbc/pgjdbc
X-GitHub-Type: review_comment
X-GitHub-Url: https://github.com/pgjdbc/pgjdbc/pull/3257#discussion_r1609885203
Content-Type: text/plain; charset=utf-8

(on .github/workflows/main.yml:154)

There's a lot of random failures building and testing against HEAD so we should ignore errors. We'll see have the details in actions to see what happened, but no point in CI randomly turning red because HEAD itself is broken. 

That's what the omni action would do: https://github.com/pgjdbc/pgjdbc/blob/450488c142fdc368cab54e8257407603acc18c4f/.github/workflows/omni.yml#L374

Also, we should probably disable HEAD entirely for PRs and enable it always for master and any daily runs.