[Feature] Update the smoke test VSIX and remove outdated packages during release

Fechada
#2,997 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Um pull request relacionado já foi integrado.

  • #2996 de @EmmaYuan1015 — integrado

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
35/100
Tipo de issue
Funcionalidade
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
typescript, vscode
Domínio
devtools

Direção de pesquisa

Start by reading the Gulp release task and the smoke test resource directory mentioned in the issue. Trace how packaging creates the VSIX and how the task signals completion; then check how existing VSIX files are handled. Done means release waits for packaging, removes outdated packages, places the generated VSIX in the smoke test resources, and fails if no expected VSIX was generated.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement

Description

Running gulp release generates a new VSIX in the project root, but the smoke test resources may still contain an older VSIX.

When the extension version changes, this leaves multiple VSIX files and requires manually replacing the package used by smoke tests.

Additionally, the release task calls its callback before the asynchronous packaging process finishes, so Gulp does not reliably wait for completion.

Expected behavior

The release task should wait for packaging to finish, remove outdated VSIX files, and move the newly generated VSIX into extension.

No VSIX should remain in the project root after a successful release.

Proposed fix

  • Return the release Promise so Gulp waits for packaging to complete.
  • Remove existing VSIX files from the project root before packaging.
  • Remove the outdated VSIX from the smoke test resources.
  • Move the newly generated VSIX into the smoke test resources.
  • Fail the release task if the expected VSIX was not generated.
Linguagem predominante
TypeScript
Estrelas
2.7k
Forks
301
Merge médio
1d 3h
PRs com merge (30d)
30

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de microsoft/vscode-react-native

Todas as issues de microsoft/vscode-react-native

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.