Skip to content

allow for spaces in magic arguments, fixes #12729 - #13027

Closed
ivanov wants to merge 4 commits into
ipython:mainfrom
ivanov:fix-12729
Closed

allow for spaces in magic arguments, fixes #12729#13027
ivanov wants to merge 4 commits into
ipython:mainfrom
ivanov:fix-12729

Conversation

@ivanov

@ivanov ivanov commented Jun 24, 2021

Copy link
Copy Markdown
Member
  • a failing test
  • new code
  • change to tests that break with the new code, due to backwards incompatibility
  • documentation of backwards incompatible change

ivanov added a commit to noteable-io/noteable-notebook-magics that referenced this pull request Jun 24, 2021
This is a workaround until the fix lands in IPython 8.0 via
ipython/ipython#13027
@ivanov

ivanov commented Jun 25, 2021

Copy link
Copy Markdown
Member Author

Windows 💥

@bollwyvl

Copy link
Copy Markdown
Contributor

Windows 💥

every dang time, man

Comment thread IPython/core/tests/test_magic_arguments.py Outdated
@MrMino

MrMino commented Jun 25, 2021

Copy link
Copy Markdown
Member

I have a feeling that posix=True might break under Windows

I didn't realize strict=False makes it fallback on the non-win32 implementation, I seriously thought this would be an ignored flag on win32 😄.

Does it actually work? I have no way of checking how the call stack looks like, I don't have any win32 on hand 😛.

Edit: oh, right, so the appveyor thing is actually about the win32 test, not intermittent. Nvm then.

Co-authored-by: Blazej Michalik <6691643+MrMino@users.noreply.github.com>
@MrMino MrMino added the magics label Jun 28, 2021
ivanov added a commit to noteable-io/noteable-notebook-magics that referenced this pull request Jun 29, 2021
This is a workaround until the fix lands in IPython 8.0 via
ipython/ipython#13027
ivanov added a commit to noteable-io/noteable-notebook-magics that referenced this pull request Jun 30, 2021
This is a workaround until the fix lands in IPython 8.0 via
ipython/ipython#13027
ivanov added a commit to noteable-io/noteable-notebook-magics that referenced this pull request Jun 30, 2021
This is a workaround until the fix lands in IPython 8.0 via
ipython/ipython#13027
@Carreau

Carreau commented Feb 23, 2025

Copy link
Copy Markdown
Member

Closing old stale PRs, sorry if this was not addressed, but this has been a few years, so there are few chances of someone actually pushing this through.

Feel free to resubmit if you feel this has a chance to be revived.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants