report_commands.js runtime output still says 'server' in four places; one names the destination

Aperta Adatta ai principianti
#2,203 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
cli

Direzione di ricerca

Leggi src/core/cli/report_commands.js intorno alla riga 935 e individua l’aspettativa sull’output di report-render che riguarda quella frase. Riformula solo la frase del Markdown generato affinché usi la terminologia di destinazione concordata, lasciando invariate le altre tre menzioni del server; esegui il test report-render pertinente e conferma che l’output previsto è stato aggiornato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Source: #2195 at ffc56a5d71f5cc51dd32fb9da5b3fa85f8787665
Location: src/core/cli/report_commands.js:778,796,935,1165
Severity: low

Evidence

Verified at ffc56a5d: :778 and :796 say 'is the server up to date?' (naming the peer software), :1165 says 'ask a server admin' (naming a role), and :935 emits generated report Markdown saying 'They ran on the server over the whole org' (naming the destination). None are help surfaces, so the #2189 guard does not and should not reach them.

Why deferral is safe

Outside issue #2189's acceptance condition (help surfaces only). Three of the four uses are correct as-is because they name the peer software or a role, not the destination; only the :935 generated-Markdown sentence names the destination in the non-settled vocabulary. Smallest-change rule: left alone in this PR, recorded here.

Acceptance condition

A follow-up rewords only the :935 generated-Markdown sentence to the settled destination vocabulary (e.g. 'They ran on the remote over the whole org'), leaves :778, :796, and :1165 unchanged, and updates any report-render output expectation that covers that sentence.

Review preference; backlog only. A human may explicitly delegate it with neutral:fix.

Lingua principale
JavaScript
Stelle
11
Fork
0
Merge medio
2h 25m
PR unite (30g)
311

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di hyparam/hypaware

Tutte le issue di hyparam/hypaware

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.