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

retrofit does not accommodate requirements defined in manifest.yml [BUG] #6

Closed
mattbathje opened this issue Jul 19, 2023 · 0 comments · Fixed by #12
Closed

retrofit does not accommodate requirements defined in manifest.yml [BUG] #6

mattbathje opened this issue Jul 19, 2023 · 0 comments · Fixed by #12
Assignees
Labels
bug Something isn't working

Comments

@mattbathje
Copy link

Description

When converting a mycroft skill, if the python library requirements were defined in manifest.yml (instead of requirements.txt), they are not carried over into the requirements.txt file.

Steps to Reproduce

Using the --retrofit option, attempt to convert a mycroft skill that has python requirements defined in manifest.yml

Relevant Code

Example manifest.yml contents:

dependencies:
    python:
        - akinator
        - rapidfuzz

Other Notes

No response

@mattbathje mattbathje added the bug Something isn't working label Jul 19, 2023
@mikejgray mikejgray self-assigned this Aug 25, 2023
mikejgray added a commit that referenced this issue Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants