Skip to content

move trusted configs from cache to config dir#836

Merged
jdx merged 1 commit intomainfrom
migrate-trusted-configs
Aug 23, 2023
Merged

move trusted configs from cache to config dir#836
jdx merged 1 commit intomainfrom
migrate-trusted-configs

Conversation

@jdx
Copy link
Copy Markdown
Owner

@jdx jdx commented Aug 23, 2023

also made them symlinks so you can see where they go

@jdx jdx changed the title migrate trusted configs move trusted configs from cache to config dir Aug 23, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 23, 2023

Codecov Report

Patch coverage: 86.95% and project coverage change: -0.11% ⚠️

Comparison is base (030dd18) 87.72% compared to head (248109f) 87.61%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #836      +/-   ##
==========================================
- Coverage   87.72%   87.61%   -0.11%     
==========================================
  Files         132      132              
  Lines       11521    11534      +13     
==========================================
- Hits        10107    10106       -1     
- Misses       1414     1428      +14     
Files Changed Coverage Δ
src/migrate.rs 57.57% <72.72%> (+7.57%) ⬆️
src/config/config_file/mod.rs 94.44% <100.00%> (+0.07%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jdx jdx force-pushed the migrate-trusted-configs branch from 30594dd to a0148e9 Compare August 23, 2023 16:22
also made them symlinks so you can see where they go
@jdx jdx force-pushed the migrate-trusted-configs branch from a0148e9 to 248109f Compare August 23, 2023 16:36
@jdx jdx enabled auto-merge (squash) August 23, 2023 16:37
@jdx jdx merged commit 4a17143 into main Aug 23, 2023
@jdx jdx deleted the migrate-trusted-configs branch August 23, 2023 16:52
jdx pushed a commit that referenced this pull request Apr 9, 2024
Co-authored-by: James Hegedus <jthegedus@hey.com>
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