Document fetch command, fix search index, add type-check CI, structured error handling#55
Closed
ctate wants to merge 2 commits into
Closed
Document fetch command, fix search index, add type-check CI, structured error handling#55ctate wants to merge 2 commits into
ctate wants to merge 2 commits into
Conversation
…ed error handling - Document `opensrc fetch` in CLI README and docs site commands page - Add /auth to allDocsPages so Authentication is searchable - Add `pnpm type-check` step to docs CI job - Replace Box<dyn Error> with thiserror enum across the CLI - Fix get_opensrc_dir() to return Result instead of calling process::exit(1)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
opensrc fetch— added to CLI README and docs site commands page (shipped in 0.7.2 but was missing from both)/authtoallDocsPagesso the Authentication page is discoverable via site searchpnpm type-checknow runs in the docs CI jobBox<dyn std::error::Error>with athiserror-basedErrorenum across the entire CLI, eliminating theprocess::exit(1)inget_opensrc_dir()and giving every error site a typed variant