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

'Verify All' option should skip '-skipped' problems #27

Closed
gdfelt opened this issue May 17, 2019 · 4 comments
Closed

'Verify All' option should skip '-skipped' problems #27

gdfelt opened this issue May 17, 2019 · 4 comments

Comments

@gdfelt
Copy link
Contributor

gdfelt commented May 17, 2019

No description provided.

@iKevinY
Copy link
Owner

iKevinY commented May 19, 2019

Hmm, what's the current behaviour of --verify-all? Reading through my documentation it seems like I accounted for this, but maybe I didn't (it's been a looong time since I've looked at EulerPy 😅).

@gdfelt
Copy link
Contributor Author

gdfelt commented May 19, 2019 via email

@iKevinY iKevinY added the bug label May 21, 2019
@iKevinY
Copy link
Owner

iKevinY commented May 21, 2019

Hmm, yeah I agree the behaviour should be to skip a skipped problem, even if it's actually correct. I should have some bandwidth to fix this over the next week or so.

@iKevinY
Copy link
Owner

iKevinY commented Dec 14, 2019

Sorry about the delay, this slipped my mind since I haven't hacked on EulerPy in a while. Thinking about it a bit harder, I think it actually makes sense for --verify-all to attempt to run everything, stripping the suffix on correct, skipped problems.

The use case I imagine is that users go through and skip a bunch of problems, and go back and solve some of the skipped ones; --verify-all gives them a quick way to go through and "unskip" them en masse.

@iKevinY iKevinY removed the bug label Dec 14, 2019
@iKevinY iKevinY closed this as completed Dec 14, 2019
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

No branches or pull requests

2 participants