Skip to content

Cache schema and improved exception handling - #191

Merged
chrisarridge merged 2 commits into
mainfrom
ca/cache-schema
Oct 19, 2026
Merged

chrisarridge merged 2 commits into
mainfrom
ca/cache-schema

Conversation

@chrisarridge

@chrisarridge chrisarridge commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

This PR enables caching on schema fetches so that the application doesn't crash if it cannot fetch 360Giving schema from external servers. It also improves error handling.

Use the force_cache option in libcove to force caching of 360 schema
so if schema URLs are temporarily unavailable the library can still
function.  If the requesting the schema fails for any reason we also
add some exception handling to improve debugging and logging of errors.
This closes #190.
@chrisarridge chrisarridge changed the title Ca/cache schema Cache schema and improved exception handling Oct 12, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 38243327494

Coverage remained the same at 81.013%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 79
Covered Lines: 64
Line Coverage: 81.01%
Coverage Strength: 0.81 hits per line

💛 - Coveralls

@chrisarridge
chrisarridge merged commit 6535597 into main Oct 19, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants