deno: CI runs one Deno version, and a matrix reference is dead
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 68/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- deno, typescript
- Domaine
- ci-cd, testing-qa
Piste de recherche
Commencez par .github/workflows/build.yml:548 et dev-packages/deno-integration-tests/suites/orchestrion-mysql/test.ts:19. Exécutez le job d’intégration Deno concerné sur Deno 2.9 et vérifiez le comportement de --import/--preload. C’est terminé lorsque le workflow ne contient plus de référence de matrice obsolète et que le TODO obsolète a été mis à jour ou supprimé, les tests d’intégration passant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
.github/workflows/build.yml:548:
deno-version: ${{ matrix.deno-version || 'v2.8.3' }}
The job has no strategy: matrix: block, so matrix.deno-version is always empty and the version is always v2.8.3. It reads like a matrix that was planned and never wired up.
Related stale note in dev-packages/deno-integration-tests/suites/orchestrion-mysql/test.ts:19:
// The orchestrion runtime hook only works as a FIRST import inside the entry
// graph in Deno 2.8.0 through 2.8.2.
// TODO: revisit a `--import` or `--preload` approach once Deno 2.8.3 ships.
Deno 2.9 is out. The TODO is due.
Work item. Either add the matrix the expression expects, or drop the dead matrix.deno-version ||. Separately, re-check the --import / --preload path on 2.9 and update or delete the TODO.
**Prior art **(related history only). No issue. getsentry/sentry-javascript#22790 (merged) set the >= 2.8.3 floor, and getsentry/sentry-javascript#23256 (merged) bumped the setup-deno action. Neither touched the dead matrix.deno-version expression.
- Langage dominant
- TypeScript
- Étoiles
- 8.7k
- Forks
- 1.9k
- Merge moyen
- 1 j 16 h
- PR mergées (30 j)
- 576
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de getsentry/sentry-javascript
-
Browser Bug Next.js Traces Waiting for: Product Owner
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
getsentry/sentry-javascript#24672 · 1 commentaire ·
-
javascript
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
getsentry/sentry-javascript#24200 · 2 commentaires ·
-
javascript Task
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
getsentry/sentry-javascript#24134 · 1 commentaire ·
-
Cloudflare Workers javascript Tests
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
getsentry/sentry-javascript#24051 · 1 commentaire ·
-
Bug Bun javascript
Difficulté 2/5 1-3 heures Accessibilité débutants 92/100
getsentry/sentry-javascript#24045 · 1 commentaire ·
Toutes les issues de getsentry/sentry-javascript
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
bcgov/bc-wallet-mobile#4761 · 1 commentaire ·
-
external-issue to-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
area-deployment area-integrations triage:bot-seen
Difficulté 2/5 Une demi-journée Accessibilité débutants 86/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
refactor
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100