use github continue-on-error to avoid red CI with test-pydantic-integration

[davidhewitt]

Change Summary

At the moment CI is red on all runs because of test-pydantic-integration. I believe that using GHA's continue-on-error setting will have the same functionality as the alls-green check however will also mean the PR gets a green tick.

Related issue number

N/A

Checklist

• Unit tests for the changes exist
• Documentation reflects the changes where applicable
• Pydantic tests pass with this pydantic-core (except for expected changes)
• My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @dmontagu

[codecov]

Codecov Report

Merging #817 (f4a29c2) into main (f89c8b2) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #817   +/-   ##
=======================================
  Coverage   93.50%   93.50%           
=======================================
  Files         102      102           
  Lines       14587    14587           
  Branches       25       25           
=======================================
  Hits        13639    13639           
  Misses        942      942           
  Partials        6        6           

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f89c8b2...f4a29c2. Read the comment docs.

[codspeed-hq]

CodSpeed Performance Report

Merging #817 will not alter performance

_{Comparing dh/pydantic-integration (f4a29c2) with main (f89c8b2)}

Summary

✅ 135 untouched benchmarks

[davidhewitt]

Looks like the PR still has a red cross, but the actions CI status is successful:

https://github.com/pydantic/pydantic-core/actions/runs/5645640678

[davidhewitt]

More importantly, people won't get emails notifying of CI run failure.

[davidhewitt]

please review

[samuelcolvin]

LGTM

[samuelcolvin]

doesn't seem to make that much difference? We still get the cross, so the PR still shows as failing.

[davidhewitt]

doesn't seem to make that much difference? We still get the cross, so the PR still shows as failing.

As per comments above, the red cross is still there but the actions run succeeds so no email notifying of failure.