Skip to content

feat: update accounts:set to work with keychain managers#3696

Open
k80bowman wants to merge 4 commits intofeat/credential-mgr-integrationfrom
k80/accounts-set
Open

feat: update accounts:set to work with keychain managers#3696
k80bowman wants to merge 4 commits intofeat/credential-mgr-integrationfrom
k80/accounts-set

Conversation

@k80bowman
Copy link
Copy Markdown
Contributor

@k80bowman k80bowman commented May 8, 2026

Summary

Updates the accounts:set command to work with keychain managers.

Type of Change

Breaking Changes (major semver update)

  • Add a ! after your change type to denote a change that breaks current behavior

Feature Additions (minor semver update)

  • feat: Introduces a new feature to the codebase

Patch Updates (patch semver update)

  • fix: Bug fix
  • deps: Dependency upgrade
  • revert: Revert a previous commit
  • chore: Change that does not affect production code
  • refactor: Refactoring existing code without changing behavior
  • test: Add/update/remove tests

Testing

Notes:

  1. Check out this branch and run npm i && npm run build
  2. Run ./bin/run login to log in to your first account.
  3. Run heroku accounts:add account1 to add this account to your accounts cache for .netrc
  4. Logout of this account in the browser and then log in to a second account
  5. Run ./bin/run login to log in to your second account
  6. Run heroku accounts:add account2 to add this second account to your accounts cache for .netrc

Steps:

  1. Run ./bin/run accounts:set <username for account 1>
  2. Run ./bin/run apps and you should see the apps for account 1
  3. Run ./bin/run accounts:set <username for account 2>
  4. Run ./bin/run apps and you should see the apps for account 2
  5. Run HEROKU_NETRC_WRITE=true ./bin/run accounts:set account1
  6. Run HEROKU_NETRC_WRITE=true ./bin/run apps and you should see the apps for account 1
  7. Run HEROKU_NETRC_WRITE=true ./bin/run accounts:set account2
  8. Run HEROKU_NETRC_WRITE=true ./bin/run apps and you should see the apps for account 2

Screenshots (if applicable)

Related Issues

GUS work item: W-20867209

@k80bowman k80bowman requested a review from a team as a code owner May 8, 2026 16:25
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 16:25 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 16:25 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 16:25 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 16:25 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 17:57 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 17:57 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 17:57 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 17:57 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 21:04 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 21:04 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 21:04 — with GitHub Actions Inactive
@k80bowman k80bowman temporarily deployed to AcceptanceTests May 8, 2026 21:04 — with GitHub Actions Inactive
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