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

Add ruamel.yaml as requirement #72

Merged

Conversation

gladhorn
Copy link
Contributor

Without ruamel.yaml installed, MkDoxy doesn't work.

File ".../MkDoxy/mkdoxy/utils.py", line 5, in
from ruamel.yaml import YAML
ModuleNotFoundError: No module named 'ruamel'

Without ruamel.yaml installed, MkDoxy doesn't work.

File ".../MkDoxy/mkdoxy/utils.py", line 5, in <module>
    from ruamel.yaml import YAML
ModuleNotFoundError: No module named 'ruamel'
@gladhorn
Copy link
Contributor Author

I'm not sure what's best, should this be versioned or not? In any case, nothing works for me without the yaml package.

@tasansal
Copy link

This is a duplicate of #71

The consensus was to use PyYAML instead and drop ruamel.yaml dependency.

@JakubAndrysek JakubAndrysek merged commit 0ce79b6 into JakubAndrysek:main Jul 19, 2023
1 check passed
@JakubAndrysek
Copy link
Owner

I have merged this PR with ruamel.yaml for successful work of this plugin.
If anyone adds PR with better PyYAML I will merge it to the master.

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