cli: rename experimental login environment variable
Metadata
Current evaluation
Merged in commit 88e4f922f, renaming experimental login variable SNAPCRAFT_LOGIN to SNAPCRAFT_STORE_AUTH to avoid conflicts with CI/CD environment names.
Resolution: closed resolved
Reason: PR #3493 was approved and merged. Repository history confirms commit 88e4f922f, which replaced SNAPCRAFT_LOGIN with SNAPCRAFT_STORE_AUTH.
Evaluated by: mr-cal-bot
No scores available.
Issue body
It was SNAPCRAFT_LOGIN, which conflicted with commonly named
environment variables used for CI/CD.
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in commit 88e4f922f, renaming experimental login variable SNAPCRAFT_LOGIN to SNAPCRAFT_STORE_AUTH to avoid conflicts with CI/CD environment names. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #3493 in commit 88e4f922f, renaming SNAPCRAFT_LOGIN to avoid conflicts with commonly named CI/CD environment variables. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged renaming the experimental login environment variable from SNAPCRAFT_LOGIN to avoid CI/CD conflicts. Approved by reviewer, passed CI, and integrated with a two-line change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Renamed the experimental login environment variable from SNAPCRAFT_LOGIN to prevent conflicts with standard CI/CD variables. The change passed code coverage checks and was approved by a maintainer. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1922815 Error: Invalid value for '--experimental-login' | snapcraft (launchpad) | closed | SNAPCRAFT_LOGIN env var clashed with the --experimental-login flag's Click boolean option, breaking CI logins. Fixed by PR #3493 (commit 88e4f922f), renaming the variable to SNAPCRAFT_STORE_AUTH, shipped in Snapcraft 4.6.1. | |
| #6098 fix: Tweak error message about SNAPCRAFT_STORE_CREDENTIALS | snapcraft | merged | Merged in commit ee69b79b1, clarifying that login cannot be used while SNAPCRAFT_STORE_CREDENTIALS is set and removing ambiguous wording. | |
| #3889 cli: add legacy envvar for login and check stdin | snapcraft | merged | Merged into hotfix/7.1.3 as 72ad273f2. Adds legacy login envvar plus a closed-stdin check so builds work when Launchpad stops logging, and enables SNAPCRAFT_ENABLE_DEVELOPER_DEBUG for core22 (CRAFT-1231). |