Skip to content

Remove bt config#278

Open
Cedric / ViaDézo1er (viadezo1er) wants to merge 8 commits into
mainfrom
cedric/remove-dead-code-bt-config
Open

Remove bt config#278
Cedric / ViaDézo1er (viadezo1er) wants to merge 8 commits into
mainfrom
cedric/remove-dead-code-bt-config

Conversation

@viadezo1er

@viadezo1er Cedric / ViaDézo1er (viadezo1er) commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

bt config was a command to edit the config.json files in ~/.config/bt and .bt in git projects, but it has never been used: since it was introduced in 09e5041, it has always been commented in src/main.rs.

src/config/mod.rs has been transformed into a helper used for authentication by the switch, status, auth, init and sync subcommands. As such, I think moving it to src/config.rs makes the most sense.

Based on cedric/prune-profiles from #274, will be merged after it.
For reviews, don't forget to compare against cedric/prune-profiles and not against main.

first draft

BREAKING-CHANGE: --profile and --prefer-profile removed
Breaks any script using them
…n to use the new auth storage format orgid,email
The field is needed by OAuth but it's unused both bt and the backend
Therefore exposing it is useless
Now bt_cli is hardcoded as its value
used to edit the config.json files
bt switch has replaced its role
config.mod.rs is still useful
@github-actions

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit 350d8af065a0:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er Cedric / ViaDézo1er (viadezo1er) changed the title cedric/remove dead code bt config Remove bt config Jul 17, 2026
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review July 17, 2026 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant