Skip to content

Navigation Menu

Appearance settings

GitHub Copilot
Write better code with AI

GitHub Spark New
Build and deploy intelligent apps

GitHub Models New
Manage and compare prompts

GitHub Advanced Security
Find and fix vulnerabilities

Actions
Automate any workflow
Codespaces
Instant dev environments

Issues
Plan and track work

Code Review
Manage code changes

Discussions
Collaborate outside of code

Code Search
Find more, search less
Explore

Why GitHub

Documentation

GitHub Skills

Blog
Integrations

GitHub Marketplace

MCP Registry
View all features
By company size

Enterprises

Small and medium teams

Startups

Nonprofits
By use case

App Modernization

DevSecOps

DevOps

CI/CD

View all use cases
By industry

Healthcare

Financial services

Manufacturing

Government

View all industries
View all solutions
Topics

AI

DevOps

Security

Software Development

View all
Explore

Learning Pathways

Events & Webinars

Ebooks & Whitepapers

Customer Stories

Partners

Executive Insights
GitHub Sponsors
Fund open source developers
The ReadME Project
GitHub community articles
Repositories

Topics

Trending

Collections
Enterprise platform
AI-powered developer platform
Available add-ons

GitHub Advanced Security
Enterprise-grade security features

Copilot for business
Enterprise-grade AI features

Premium Support
Enterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

SwiftPackageIndex / SwiftPackageIndex-Server Public

Notifications You must be signed in to change notification settings
Fork 50
Star 639

Code
Issues 67
Pull requests
Discussions
Actions
Projects 1
Wiki
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Wiki
Security
Insights

Rework error handling#3549

Merged

finestructure merged 33 commits intomainSwiftPackageIndex/SwiftPackageIndex-Server:mainfrom

rework-error-handlingSwiftPackageIndex/SwiftPackageIndex-Server:rework-error-handlingCopy head branch name to clipboard

Dec 19, 2024

Conversation Commits33 (33)Checks Files changed

Merged

Rework error handling#3549
finestructure merged 33 commits intomainfrom
rework-error-handling

Commits

Commits on Dec 19, 2024

Expand test case to ensure we're advancing to analysis in case of an ingestion error
finestructure
committed
Temporarily disable new check
finestructure
committed
Move existing ingestion logic into ingestOriginal
finestructure
committed
Make s3 readme functions throws(S3ReadmeError)
finestructure
committed
wip
finestructure
committed
Add back getFork
finestructure
committed
Add warning about S3ReadmeError → S3Store.Error
finestructure
committed
Add id, underlyingError to Ingestion.Error
finestructure
committed
Avoid repository.package.id!
finestructure
committed
Update recordError to handle Ingestion.Error
finestructure
committed
Ingestion specific updatePackage - all tests pass
finestructure
committed
Update test_ingest_unique_owner_name_violation to reflect new error handling
finestructure
committed
Disentangle updatePackage/recordError into Common, Analyze, Ingestion parts
finestructure
committed
Remove Common.updatePackage, clean up
finestructure
committed
Move things around
finestructure
committed
Cleanup
finestructure
committed
Add protocol ProcessingError
finestructure
committed
Address warnings in Common
finestructure
committed
Extended test_ingest_continue_on_error now passing
finestructure
committed
S3ReadmeError → S3Readme.Error
finestructure
committed
Add run { } throwing: { }
finestructure
committed
Ingestion.ingestNew → Ingestion.ingest
finestructure
committed
Rename run(_:throwing:) to run(_:rethrowing), bundle with run(_:defer:)
finestructure
committed
Cleanup
finestructure
committed
Restore async let in fetchMetadata
finestructure
committed
Cleanup
finestructure
committed
Re-enable test
finestructure
committed
Explicitly analyse packages and verify processing stage
finestructure
committed
Turn `Ingestion.fetchMetadata` into `throws(Github.Error)`
finestructure
committed
Add error forcing mechanics
finestructure
committed
Improve error logging
finestructure
committed
Update error message expectation
finestructure
committed
Make IngestCommand.run non-throwing
finestructure
committed

Footer

© 2025 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.