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.

Sign up for GitHub

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

Jump to bottom

MarkDownFile not accepting absolute file path anymore #81

Closed
KarstenSchaeffer opened this issue Feb 16, 2023 · 2 comments
Closed

MarkDownFile not accepting absolute file path anymore #81

KarstenSchaeffer opened this issue Feb 16, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@KarstenSchaeffer
Copy link

KarstenSchaeffer commented Feb 16, 2023 •
edited
Loading

Describe the bug
With "Modify tests to fix flakiness" MarkDownFile.init() has been modified to use os.getcwd() as base directory.
This makes it impossible to specify an absolute path via MdUtils.create_md_file().

To Reproduce

md_file = mdutils.MdUtils(file_name="<absolute file path>", title="My title")
md_file.create_md_file()

leads to

FileNotFoundError: [Errno 2] No such file or directory: '<current directory>/<absolute file path>'

Expected behavior
MD file is created with given absolute file_name.

Desktop (please complete the following information):

  • OS: Ubuntu
  • Version 22.04
@KarstenSchaeffer KarstenSchaeffer added the bug Something isn't working label Feb 16, 2023
@didix21
Copy link
Owner

didix21 commented Feb 16, 2023

Thanks for reporting the issue I'll check for the problem, I recommend to downgrade to 1.4.0 version.

@lauritsriple
Copy link

I got the same issue. Downgrading to 1.4.0 solves this temporarily.

This was referenced Mar 7, 2023
Closed
Closed
Merged
Merged
didix21 added a commit that referenced this issue Mar 9, 2023
@didix21
Fix #81: Absolute path are broken
75438ce
didix21 added a commit that referenced this issue Mar 10, 2023
@didix21
Fix #81: Absolute path are broken
6ab403b
didix21 added a commit that referenced this issue Mar 11, 2023
@didix21
Fix #81: Absolute path are broken
ea54a4a 
- Add new test for improving coverage.
- Install pytest on github actions.
- Replace unittest for pytest on github actions.
didix21 added a commit that referenced this issue Mar 11, 2023
@didix21
Fix #81: Absolute path are broken
df64ebb 
- Add new test for improving coverage.
- Install pytest on github actions.
- Replace unittest for pytest on github actions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allows filename to be an absolute path alexiswl/mdutils
3 participants
@lauritsriple @didix21 @KarstenSchaeffer