Skip to content

bitwarden-cli: Persist data#6462

Closed
Ra2-IFV wants to merge 1 commit into
ScoopInstaller:masterfrom
Ra2-IFV:feat_bitwarden-cli_add-persist
Closed

bitwarden-cli: Persist data#6462
Ra2-IFV wants to merge 1 commit into
ScoopInstaller:masterfrom
Ra2-IFV:feat_bitwarden-cli_add-persist

Conversation

@Ra2-IFV

@Ra2-IFV Ra2-IFV commented Jan 16, 2025

Copy link
Copy Markdown
Contributor

Reopen for #6219
See bitwarden/cli#117

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

@github-actions

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

bitwarden-cli

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

@HUMORCE

HUMORCE commented Apr 16, 2025

Copy link
Copy Markdown
Member

Will the dir creation make users open a empty database without a migration?

@Ra2-IFV

Ra2-IFV commented Apr 17, 2025

Copy link
Copy Markdown
Contributor Author

Yeah I guess so so I asked if I could write a script to migrate the data

@HUMORCE

HUMORCE commented Apr 17, 2025

Copy link
Copy Markdown
Member

If it does NOT cause the database to disappear, memo this change in notes is enought.

@Ra2-IFV

Ra2-IFV commented Apr 17, 2025

Copy link
Copy Markdown
Contributor Author

any examples of telling users how to do that? I'm not good at this

@HUMORCE

HUMORCE commented Apr 17, 2025

Copy link
Copy Markdown
Member

It might look like:

"notes": [
  "The data location has been persisted by Scoop since <version>",
  "Previous location: ...",
  "Current location: ..."
],

for more examples, searching "notes" on main/extras repo via VS Code.

@Ra2-IFV Ra2-IFV force-pushed the feat_bitwarden-cli_add-persist branch from 332c057 to 64fe56a Compare April 17, 2025 12:00
@Ra2-IFV

Ra2-IFV commented Apr 17, 2025

Copy link
Copy Markdown
Contributor Author

Done. Also please take a look at ScoopInstaller/Extras#15311

@HUMORCE HUMORCE left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

use variables, current version

Comment thread bucket/bitwarden-cli.json Outdated

@z-Fng z-Fng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry, you’re not able to edit this repository directly—you need to fork it and propose your changes from there instead.

Please resolve the merge conflicts. We can't make edits.

@z-Fng z-Fng added duplicate This issue or pull request already exists and removed review-needed labels May 5, 2026
@z-Fng

z-Fng commented May 5, 2026

Copy link
Copy Markdown
Member

Synced to:

@z-Fng z-Fng closed this May 5, 2026
pull Bot pushed a commit to 991548267/Scoop-Main that referenced this pull request May 5, 2026
* Relates to ScoopInstaller#6462.

Co-authored-by: IFV <96299096+Ra2-IFV@users.noreply.github.com>
@Ra2-IFV

Ra2-IFV commented May 17, 2026

Copy link
Copy Markdown
Contributor Author

Thank you. Deleting.

@Ra2-IFV Ra2-IFV deleted the feat_bitwarden-cli_add-persist branch May 17, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants