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

Setting ans to a expression in REPLBlocks. #926

Merged
merged 2 commits into from
Jan 24, 2019
Merged

Setting ans to a expression in REPLBlocks. #926

merged 2 commits into from
Jan 24, 2019

Conversation

fredrikekre
Copy link
Member

@fredrikekre fredrikekre commented Jan 23, 2019

Fix #923

@fredrikekre fredrikekre requested a review from mortenpi January 23, 2019 14:08
@mortenpi mortenpi changed the title fix #923: setting ans to a expression in REPLBlocks. Setting ans to a expression in REPLBlocks. Jan 23, 2019
@mortenpi mortenpi added this to the 0.21.1 milestone Jan 23, 2019
@mortenpi
Copy link
Member

Seems to work, so LGTM. It might be good to test against Base doctests before the release, just in case.

Could use a CHANGELOG entry, but were thinking about adding them later all at once?

@mortenpi
Copy link
Member

Also, it might be good to drop the previously failing example into test/examples.

@fredrikekre
Copy link
Member Author

Done.

@mortenpi mortenpi merged commit 8d17f68 into master Jan 24, 2019
@mortenpi mortenpi deleted the fe/repl-block branch January 24, 2019 01:11
mortenpi pushed a commit that referenced this pull request Jan 27, 2019
* fix #923: setting ans to a expression in REPLBlocks.

(cherry picked from commit 8d17f68)
@mortenpi mortenpi mentioned this pull request Jan 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants