Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: API edit command; apis.json schema #208

Merged
merged 1 commit into from
Aug 1, 2023
Merged

Conversation

danielgtaylor
Copy link
Owner

Add a basic restish api edit command to edit the API config file now that it's in different locations depending on the operating system.

Also adds a schema for the apis.json config file, making it easier to edit in VSCode and other editors.

@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #208 (c95742f) into main (3a541b4) will increase coverage by 0.67%.
Report is 4 commits behind head on main.
The diff coverage is 93.75%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
+ Coverage   75.37%   76.04%   +0.67%     
==========================================
  Files          26       26              
  Lines        3513     3624     +111     
==========================================
+ Hits         2648     2756     +108     
  Misses        666      666              
- Partials      199      202       +3     
Files Changed Coverage Δ
cli/apiconfig.go 83.80% <93.75%> (+3.62%) ⬆️

... and 3 files with indirect coverage changes


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 3a541b4...c95742f. Read the comment docs.

@danielgtaylor danielgtaylor merged commit 2242e20 into main Aug 1, 2023
@danielgtaylor danielgtaylor deleted the api-edit-cmd branch August 1, 2023 23:58
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.

1 participant