JustinTimperio / pacback

Advanced Version Control for Arch Linux
MIT License
148 stars 4 forks source link

Feature Request: Add "List all restore points" as an option #12

Closed Gr3q closed 4 years ago

Gr3q commented 4 years ago

What is says in the title. Maybe using -l?

JustinTimperio commented 4 years ago

I'm planning on adding this soon along with a remove function. (maybe -del or -rm) I have been working on a pretty major overhaul, so this should be simple once everything is working. If you want to see the status of this, checkout to the refactor fork. I still need to do a lot more unit testing before I merge it though.

JustinTimperio commented 4 years ago

Okay so I just added this feature in the refactor fork. I will try and merge this branch soon

Gr3q commented 4 years ago

Thanks, I can wait until the merge

JustinTimperio commented 4 years ago

Hey Gr3q, I'm looking for some help testing my refactor before I merge it. If you have any interest in doing so let me know.

Thanks

Gr3q commented 4 years ago

Sure, I can do that, let me know what to test for and I'll do it

JustinTimperio commented 4 years ago

Thanks, I appreciate the help. Version 1.6 is a massive refactor from 1.5. While the commands are the same, under the hood things are far more refined from a code perspective. Right now I'm putting the final touches on the update before I merge.

Gr3q commented 4 years ago

I can do point number 2, 3, 4 on 2 computers quite easily. Number 5 (and maybe 1?) sounds like it could use unit tests rather than manual testing tho.

For number 1 and 6 you will have to specify what to test for and I will do them.

JustinTimperio commented 4 years ago

Unfortunately, most of the testing needs to be done manually. I use flake8 to look for issues but a lot of the system commands just need to be tested manually.

I have finished 1, 3, 4, 5, 6

I think I'm going to merge this unless you have found any serious errors I have missed.

JustinTimperio commented 4 years ago

Since the list feature is now added I'm going to close this.