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

feat(jans-cli-tui): update to prompt_toolkit v3 #7491

Merged
merged 3 commits into from
Feb 1, 2024

Conversation

davispuh
Copy link
Contributor

Prepare


Description

Target issue

prompt_toolkit v2 is essentially unmaintained since last release was in 2019. This PR switches implementation to prompt_toolkit v3

Implementation Details

See https://python-prompt-toolkit.readthedocs.io/en/master/pages/upgrading/3.0.html


Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

@davispuh davispuh requested a review from devrimyatar as a code owner January 22, 2024 21:11
@ossdhaval
Copy link
Contributor

Hi @davispuh, Thanks for submitting this PR. The code owner will review it soon. At the same time, I wanted to request you to add DCO, guidelines here, to your commits.

@ossdhaval ossdhaval added kind-dependencies Pull requests that update a dependency file priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged comp-jans-cli-tui Component affected by issue or PR labels Jan 23, 2024
@ossdhaval ossdhaval requested a review from moabu January 23, 2024 05:41
v2 is essentially unmaintained since last release was in 2019

Signed-off-by: Dāvis Mosāns <[email protected]>
@davispuh
Copy link
Contributor Author

Added DCO.

@moabu moabu merged commit dcc3abd into JanssenProject:main Feb 1, 2024
2 checks passed
devrimyatar added a commit that referenced this pull request Feb 3, 2024
devrimyatar added a commit that referenced this pull request Feb 3, 2024
moabu pushed a commit that referenced this pull request Feb 3, 2024
* Revert "feat(jans-cli-tui): update to prompt_toolkit v3 (#7491)"

This reverts commit dcc3abd.

Signed-off-by: Mustafa Baser <[email protected]>

* Revert "fix(jans-linux-setup): update prompt-toolkit 3.0.43 (#7604)"

This reverts commit e35963f.

Signed-off-by: Mustafa Baser <[email protected]>

---------

Signed-off-by: Mustafa Baser <[email protected]>
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
v2 is essentially unmaintained since last release was in 2019

Signed-off-by: Dāvis Mosāns <[email protected]>
Co-authored-by: Mohammad Abudayyeh <[email protected]>
Co-authored-by: Devrim <[email protected]>
Former-commit-id: dcc3abd
yuriyz pushed a commit that referenced this pull request Nov 7, 2024
* Revert "feat(jans-cli-tui): update to prompt_toolkit v3 (#7491)"

This reverts commit c7f1f97 [formerly dcc3abd].

Signed-off-by: Mustafa Baser <[email protected]>

* Revert "fix(jans-linux-setup): update prompt-toolkit 3.0.43 (#7604)"

This reverts commit 13fe5a6 [formerly e35963f].

Signed-off-by: Mustafa Baser <[email protected]>

---------

Signed-off-by: Mustafa Baser <[email protected]>
Former-commit-id: b3f6834
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-cli-tui Component affected by issue or PR kind-dependencies Pull requests that update a dependency file priority-4 Minor issue or PR is not relevant to core functions, or relates to the usability of system triaged Issue or PR is fully triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants