microsoft / microsoft/vscode-azurestaticwebapps
Remove unneeded `BROWSER: 'none'` environment variable
Open
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 99
- Forks
- 39
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 2
Description
As of v1.0.0, this is now handled inside the SWA CLI here.
added in PR: https://github.com/Azure/static-web-apps-cli/pull/472
Probably a good idea to wait until after we release #710 to give users time to update to v1.0.0
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect src/cli/SwaCliTaskProvider.ts at lines 107-113 and compare the SWA CLI v1.0.0 handling linked in the issue. Confirm release #710 has shipped so users can update, then remove the obsolete BROWSER environment variable and verify the extension no longer sets it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, typescript, vscode
- Domain
- cli, tooling
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 45/100