make %%writefile work w pshnb #6
Conversation
RensDimmendaal
changed the title
| from IPython.paths import get_ipython_dir | ||
| from IPython.core.interactiveshell import InteractiveShell |
There was a problem hiding this comment.
removed some unused imports
| @@ -118,11 +114,23 @@ def bash(self, line, cell=None): | |||
| raise e from None | |||
| if disp and res: print(res) | |||
|
|
|||
| @magic_arguments() | |||
| @argument('-a', '--append', action='store_true', help='Append contents of the cell to an existing file. The file will be created if it does not exist.') | |||
There was a problem hiding this comment.
same args as the original %%writefile
|
Thanks @RensDimmendaal ! I just added: 4693623 . With this change, it is now quite easy to ensure that the bash interpreter directory is synchronized with Python's current directory, and you could do this before you run writefile. This way you have the choice as to whether you want to write a file to the current Python or current shell directory. Also, to write to the current shell directory, you can simply use normal bash cat and heredoc. I'm thinking we should leave it that way rather than shadowing the writefile magic. I'm a bit nervous that shadowing the right file magic might also cause some kind of strange bugs which I am having trouble thinking about but feel like they could get confusing. wdyt? |
Whoah! Smart! More concise and more robust. We could take this one step further:
Then:
|
%%writefile is a built-in magic that we can now also make aware of the pshnb workdir. This PR makes that possible.