-
Notifications
You must be signed in to change notification settings - Fork 0
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
Not creating the requirements.txt file #7
Comments
Hi, thanks for reporting! The fact that the plugin works once you comment out the lines proves that:
The lines you're refering to ensure that the In the PR above I tried to reproduce the problem by using tomlkit directly to parse an example file. The test passes.
Given this information, we can hopefully create a failing test case for me to address or find a formatting problem in your |
I have the same issue. Commenting the lines out wshayes mentions also fixes it for me. Thanks for the plugin :)
|
1 similar comment
I have the same issue. Commenting the lines out wshayes mentions also fixes it for me. Thanks for the plugin :)
|
Thanks @martinmoldrup. I was able to reproduce and fix the issue with the file you've provided. I've added a test for this case and relaxed the type check a bit in #8 . I'll create a new release shortly. Could you update the package and let me know if it works for you? @wshayes , I hope this solves your problem as well! |
I had the same problem as the reporter had, and I can confirm that 0.2.1 fixes the problem for me at least. |
Summary
requirements.txt file was not being auto-generated when running
poetry lock
after installing the pluginBackground:
Installed the plugin via
pipx inject poetry poetry-auto-export
Checked that the plugin was installed using
poetry self show plugins
Result:
Was not getting the requirements.txt and requirements-dev.txt files being generated when running
poetry lock
Potential fix:
When I commented out the following lines in the plugin, it worked fine. I'm not sure why that check exists.
https://github.com/Ddedalus/poetry-auto-export/blob/ce6593567a617ec88878442134b9d330310b82db/poetry_auto_export/plugin.py#L31C1-L32C27
PS - thank you so much for creating this plugin - I'd finally gotten fed up after forgetting to update this way too many times when working with docker and your plugin magically showed up when I did a search for something like this.
The text was updated successfully, but these errors were encountered: