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 --sync CLI option to update subcommand #8931

Merged
merged 8 commits into from
Feb 6, 2024

Conversation

Cypher1
Copy link
Contributor

@Cypher1 Cypher1 commented Feb 1, 2024

Adds --sync to the update subcommand to match install.

Copy link
Member

@radoering radoering left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. Can you update the docs and add a test analogous to

def test_sync_option_is_passed_to_the_installer(
tester: CommandTester, mocker: MockerFixture
) -> None:
"""
The --sync option is passed properly to the installer.
"""
assert isinstance(tester.command, InstallerCommand)
mocker.patch.object(tester.command.installer, "run", return_value=1)
tester.execute("--sync")
assert tester.command.installer._requires_synchronization
, please?

@Cypher1
Copy link
Contributor Author

Cypher1 commented Feb 2, 2024

Sounds good. Will get to it soon. Thx

@Cypher1
Copy link
Contributor Author

Cypher1 commented Feb 5, 2024

pre-commit.ci autofix

tests/console/commands/test_update.py Outdated Show resolved Hide resolved
@Cypher1
Copy link
Contributor Author

Cypher1 commented Feb 5, 2024

pre-commit.ci autofix

@radoering radoering added the impact/docs Contains or requires documentation changes label Feb 6, 2024
Copy link

github-actions bot commented Feb 6, 2024

Deploy preview for website ready!

✅ Preview
https://website-q09xsf619-python-poetry.vercel.app

Built with commit 80c282e.
This pull request is being automatically deployed with vercel-action

@radoering radoering merged commit f3bb139 into python-poetry:master Feb 6, 2024
35 checks passed
@Cypher1
Copy link
Contributor Author

Cypher1 commented Feb 6, 2024

Thanks!

@Cypher1 Cypher1 deleted the updateSync branch February 6, 2024 21:04
Copy link

github-actions bot commented Mar 8, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
impact/docs Contains or requires documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants