Skip to content

Remove bt config - #278

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

Remove bt config#278
Cedric / ViaDézo1er (viadezo1er) wants to merge 3 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.

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit 173e2b0cba93:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-x86_64-unknown-linux-musl
  • 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
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) force-pushed the cedric/remove-dead-code-bt-config branch 2 times, most recently from 699e835 to 3aa1a8d Compare July 23, 2026 20:45
used to edit the config.json files
bt switch has replaced its role
config.mod.rs is still useful
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