Skip to content
This repository was archived by the owner on Oct 24, 2025. It is now read-only.

Fix poetry 1.0.0b1 export prints to stdout by default#395

Merged
dschep merged 4 commits into
serverless:masterfrom
mokamoto12:adjust-poetry1.0.0b1-export
Aug 12, 2019
Merged

Fix poetry 1.0.0b1 export prints to stdout by default#395
dschep merged 4 commits into
serverless:masterfrom
mokamoto12:adjust-poetry1.0.0b1-export

Conversation

@mokamoto12

Copy link
Copy Markdown
Contributor

The export command for poetry 1.0.0b1 outputs to standard output if the -o option is not specified.

Modified to use spawnSync return values.

@dschep

dschep commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

Will this break older versions of Poetry? Is there any solution that works with old and new versions?

@mokamoto12

Copy link
Copy Markdown
Contributor Author

Sorry for my lack of consideration. I'll think about it a little more.
Please let me know if you have any ideas!

@mokamoto12

mokamoto12 commented Aug 12, 2019

Copy link
Copy Markdown
Contributor Author

circleci is working? I set up circleci setting in my forked repository before. Is that bad?

I corrected code for now.

@dschep

dschep commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

Did you test it with an older version? I'm just wondering if it truly outputs nothing or still has a newline? (in which case, you can just add .trim() after the .toString().

circleci is working? I set up circleci setting in my forked repository before. Is that bad?

nah, that's fine.

@mokamoto12

Copy link
Copy Markdown
Contributor Author

Did you test it with an older version? I'm just wondering if it truly outputs nothing or still has a newline? (in which case, you can just add .trim() after the .toString().

Yes, I test it with an older version and working. But I think I should add .trim() too.
I fixed it and add a test.

@dschep

dschep commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

Awesome. Thanks! I'll merge as soon as circle finishes

@dschep

dschep commented Aug 12, 2019

Copy link
Copy Markdown
Contributor

Hmm. it does seem seting up circle might have caused issues :(

@mokamoto12

Copy link
Copy Markdown
Contributor Author

Thank you for your kindness!
But I'm worried if circle job is not running. I am sorry if I have a problem. Please let me know if there is my task.

@mokamoto12

Copy link
Copy Markdown
Contributor Author

looks circle is running! I pray to pass the tests.
I will be careful to not setting up circle in a forked repository.

@dschep dschep merged commit 7d3e263 into serverless:master Aug 12, 2019
@mokamoto12 mokamoto12 deleted the adjust-poetry1.0.0b1-export branch August 13, 2019 02:46
@username-dot-com

Copy link
Copy Markdown

I'm sorry if this was posted elsewhere, but when will this get into an npm release?

miketheman added a commit to miketheman/serverless-python-requirements that referenced this pull request Feb 18, 2020
The test was introduced to test the behavior of a particular pre-release
version of poetry, prior to version 1.0.0 being released.
It has since been released on Dec 12, 2019.

Refs: https://python-poetry.org/blog/announcing-poetry-1-0-0.html
Refs: serverless#395

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
miketheman added a commit to miketheman/serverless-python-requirements that referenced this pull request Feb 18, 2020
The test was introduced to test the behavior of a particular pre-release
version of poetry, prior to version 1.0.0 being released.
It has since been released on Dec 12, 2019.

Refs: https://python-poetry.org/blog/announcing-poetry-1-0-0.html
Refs: serverless#395

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants