Skip to content

feat: add status and diff commands #76

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

Merged
merged 41 commits into from
Jan 27, 2022
Merged

feat: add status and diff commands #76

merged 41 commits into from
Jan 27, 2022

Conversation

Shadow53
Copy link
Owner

  • hoard diff <hoard> shows the files that differ for a given hoard
  • hoard status shows the general status of all hoards (in sync or not, if they should be backed up or restored, etc.)

Resolves #72.

@Shadow53 Shadow53 added the enhancement New feature or request label Jan 10, 2022
@Shadow53 Shadow53 added this to the 0.5.0 milestone Jan 10, 2022
@Shadow53 Shadow53 self-assigned this Jan 10, 2022
@netlify
Copy link

netlify bot commented Jan 10, 2022

✔️ Deploy Preview for hoard-docs ready!

🔨 Explore the source changes: 6478d0f

🔍 Inspect the deploy log: https://app.netlify.com/sites/hoard-docs/deploys/61dbdb2979da9700072ae106

😎 Browse the preview: https://deploy-preview-76--hoard-docs.netlify.app

@netlify
Copy link

netlify bot commented Jan 10, 2022

✔️ Deploy Preview for hoard-docs ready!

🔨 Explore the source changes: 7b395a5

🔍 Inspect the deploy log: https://app.netlify.com/sites/hoard-docs/deploys/61f23b06cd02c40008fe94e0

😎 Browse the preview: https://deploy-preview-76--hoard-docs.netlify.app

@Shadow53 Shadow53 marked this pull request as draft January 10, 2022 07:10
@Shadow53
Copy link
Owner Author

Shadow53 commented Jan 13, 2022

TODO:

  • Add code + tests for ignored file paths with diffs
  • Use operation logs to help determine if file was created/deleted
  • Use diff and operation logs to impl hoard status with suggestion to backup/restore

@codecov
Copy link

codecov bot commented Jan 18, 2022

Codecov Report

Merging #76 (1cafd57) into main (ffe9d74) will decrease coverage by 94.96%.
The diff coverage is n/a.

❗ Current head 1cafd57 differs from pull request most recent head 7b395a5. Consider uploading reports for the commit 7b395a5 to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##             main   #76       +/-   ##
========================================
- Coverage   94.96%     0   -94.97%     
========================================
  Files          25     0       -25     
  Lines        1510     0     -1510     
========================================
- Hits         1434     0     -1434     
+ Misses         76     0       -76     
Impacted Files Coverage Δ
src/checkers/history/last_paths.rs
src/checkers/history/operation.rs
src/command/mod.rs
src/config/builder/hoard.rs
src/config/builder/mod.rs
src/config/mod.rs
src/env_vars.rs
src/filters/ignore.rs
src/filters/mod.rs
src/hoard/mod.rs
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffe9d74...7b395a5. Read the comment docs.

@Shadow53 Shadow53 force-pushed the 72-status-diff-commands branch from 812b7a8 to d3dcdd0 Compare January 27, 2022 04:39
@Shadow53 Shadow53 marked this pull request as ready for review January 27, 2022 05:08
Ok(result)
}

/// Returns whether the given `file` has unapplied remote changes.
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy/pasted comment needs to be modified

@Shadow53 Shadow53 merged commit 98ff07e into main Jan 27, 2022
@Shadow53 Shadow53 deleted the 72-status-diff-commands branch January 27, 2022 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Status/diff commands
1 participant