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.

Sign up for GitHub

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

Jump to bottom

#291: Don't remove priority on "do" #6

Merged
merged 24 commits into from
Apr 30, 2023
Merged

#291: Don't remove priority on "do" #6

merged 24 commits into from
Apr 30, 2023

Conversation

mariabui
Copy link
Owner

mariabui and others added 24 commits April 20, 2023 23:19
@mariabui
remove unsupported codecov dependency
7d0ffe1
@mariabui
add section in configuration for do cmd to keep priority of todo items
f9fc07a
@mariabui
modify set_completed method to keep or remove priority of todo items …
8438e05 
…based on configuration
@mariabui
add tests and config files to test keeping and removing priority of a…
4e176a0 
… todo item
@mariabui
change keep_priority config to 1 to see if all checks pass
953dc67
@mariabui
add more tests for todo items without a priority and those with depen…
4e4e3cb 
…dencies
@mariabui
add test for bad keep priority switch value
92ce8c8
@mariabui
add test cases to verify keep_priority method returns correct configu…
80c2b9d 
…red value
@mariabui
modify set_completed method to retain position of priority after comp…
6c5b2dc 
…letion, adjust tests to reflect modification
@mariabui
add tests for set_completed method to verify it keeps or removes prio…
aa919d2 
…rity of a completed todo item depending on configuration
@mariabui
add tests for child todo items completion and parent todo item activa…
ff5bbbe 
…tion
@mariabui
modify set_completed method to relocate priority to before completion…
f7e8383 
… date, adjust tests to reflect modification
@mariabui
add tests for keeping priorities of multiple completed todos
f031c04
@m6parker
added test keep priority do in test_do_command.py
070c62a
@s5teixeira
added priority test functions
f7e9912
@mariabui
remove test print statement
ae58806
@s5teixeira
added custom date test for keep priority
0d0a3e8
@s5teixeira
Merge remote-tracking branch 'origin/291_group' into 291_group
d3178b7
@priscajeanpierre
do command tests
eeb0cc3
@m6parker
added custom date test in test_do_command.py
841cbdd
@m6parker
add custom date 6 test
4692bcb
@m6parker
added more to dos to todo list and recurrence2
f69599c
@priscajeanpierre
do command tests
5ff9f12
@mariabui
fix indentation of test_keep_priority_do_subtasks_force2
b756325
@mariabui mariabui merged commit 40ec806 into master Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mariabui @m6parker @s5teixeira @priscajeanpierre