Skip to content

Update good_conditional_todo_list.py #745

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 3 commits into from
Apr 22, 2022
Merged

Conversation

jmtaysom
Copy link
Contributor

This minor change updates the example in good_conditional_todo_list.py to match the expected behavior of the previous example where the completed items in the todo list had checkmarks.

The checkmark was on the wrong list items compared to the more verbose example.
@jmtaysom
Copy link
Contributor Author

Should the changes be listed in the unreleased changes section of docs/source/about/changelog.rst or would you like me to leave that for you to handle?

@rmorshea
Copy link
Collaborator

rmorshea commented Apr 21, 2022

If you could add that to the unreleased section of the changelog that would be great!

@rmorshea
Copy link
Collaborator

actually nevermind. this isn't a functional change. I'll wave that check

@rmorshea rmorshea added the flag-no-changelog Add to a pull request that does not require a changelog label Apr 21, 2022
fixed typo. It was rendering `1 or 10`  instead of `1 of 10`
@jmtaysom
Copy link
Contributor Author

jmtaysom commented Apr 22, 2022

I have another typo fix that also doesnt change any functionality.

@rmorshea rmorshea merged commit fb45e7e into reactive-python:main Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flag-no-changelog Add to a pull request that does not require a changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants