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

pass encoding=utf8 when opening files #1288

Merged
merged 1 commit into from Feb 11, 2023
Merged

pass encoding=utf8 when opening files #1288

merged 1 commit into from Feb 11, 2023

Conversation

xl0
Copy link
Contributor

@xl0 xl0 commented Feb 6, 2023

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@hugetim
Copy link
Contributor

hugetim commented Feb 7, 2023

May also solve #1276

@jph00
Copy link
Member

jph00 commented Feb 10, 2023

Do you mind resolving the conflicts? I think it's just a case of running the new nbdev_clean AFAICT.

@xl0
Copy link
Contributor Author

xl0 commented Feb 11, 2023

@jph00 , done.

Would you also like to merge fastai/nbdev-template#9 ?

@jph00 jph00 merged commit cf5e6a5 into fastai:master Feb 11, 2023
@jph00
Copy link
Member

jph00 commented Feb 11, 2023

Would you also like to merge fastai/nbdev-template#9 ?

Yes I would! :D (BTW, if you at-mention me on a PR then it goes to my inbox and won't be missed or forgotten.)

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

Successfully merging this pull request may close these issues.

None yet

3 participants