use pathlib by danieleades · Pull Request #864 · executablebooks/MyST-Parser
Navigation Menu
{{ message }}
executablebooks / MyST-Parser Public
- Notifications You must be signed in to change notification settings
- Fork 229
Draft
danieleades wants to merge 2 commits intoexecutablebooks:masterfrom
Draft
use pathlib#864
danieleades wants to merge 2 commits intoexecutablebooks:masterfrom
use pathlib#864
danieleades wants to merge 2 commits intoexecutablebooks:masterfrom
Conversation
daniel.eades and others added 2 commits
January 26, 2024 14:55
Copy link Copy Markdown
Codecov Report
All modified and coverable lines are covered by tests ✅
Project coverage is 90.06%. Comparing base (
e74769c) to head (cb24054).
Additional details and impacted files
@@ Coverage Diff @@ ## master #864 +/- ## ======================================= Coverage 90.06% 90.06% ======================================= Files 24 24 Lines 3372 3372 ======================================= Hits 3037 3037 Misses 335 335
| Flag | Coverage Δ | |
|---|---|---|
| pytests | 90.06% <ø> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.
Copy link Copy Markdown
Member
chrisjsewell
commented
Mar 26, 2024
chrisjsewell commented
Mar 26, 2024heya @danieleades do you wanna fix the last few lints?:
myst_parser/inventory.py:410:10: PTH123 `open()` should be replaced by `Path.open()`
myst_parser/inventory.py:471:14: PTH123 `open()` should be replaced by `Path.open()`
myst_parser/mdit_to_docutils/base.py:1211:17: PTH118 `os.path.join()` should be replaced by `Path` with `/` operator
myst_parser/mdit_to_docutils/sphinx_.py:68:17: PTH118 `os.path.join()` should be replaced by `Path` with `/` operator
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants