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

Follow up items #52

Open
dfinke opened this issue May 10, 2021 · 2 comments
Open

Follow up items #52

dfinke opened this issue May 10, 2021 · 2 comments

Comments

@dfinke
Copy link
Owner

dfinke commented May 10, 2021

@jhoneill

utf8NoBOM

Doesn't run on PS 5.1. Not sure if this is the only PS 5 stopper.
I'm ok if this is not backward compatible.

ConvertToPowerShellNoteBook.ps1

It only handles basic comments in a PS1 file for conversion to a ipynb. Not sure what the effort or how to upgrade it so comments in a function, at the end of a line won't get emitted as a markdown cell.

  • nice to have
@jhoneill
Copy link
Contributor

Not sure. When it was just UTF8 the ipynb would not open in VS code, it gave an error something like "bad token at position 0"
I'm fairly sure UTF8 (with BOM) opens elsewhere - including Azure data studio where one can use Windows PowerShell without installing .Net interactive. If that's the case we (cough... I) should probably file a bug against .Net Interactive because they shouldn't fail to process files which other things process successfully. Let's avoid compat issues if practical.

@dfinke
Copy link
Owner Author

dfinke commented May 10, 2021

I believe the notebook will open. The ConvertTo-PowerShellNoteBook function will fail to run on PS 5 though.

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