Refactor: reduce copy-paste boilerplate in ec2 run-instances _fix_args

Aperta Adatta ai principianti

#10,562 aperta il 16/8/2026 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 2/5
Tempo stimato: 1-3 ore
Idoneità per principianti: 84/100

Tipo di issue: Refactoring
Chiarezza: Specificata chiaramente
Stato di attività: Tranquilla
Stack tecnologico: python
Ambito: cli

Direzione di ricerca

Apri awscli/customizations/ec2/runinstances.py ed esamina _fix_args intorno alle righe 120-141. Confronta gli spostamenti ripetuti dei parametri scalari con la trasformazione separata di PrivateIpAddress, quindi esegui il refactoring solo dei casi semplici di rinomina e spostamento in una mapping e un ciclo. Il lavoro è completato quando il codice boilerplate è ridotto, mentre il comportamento dei parametri rimane invariato, incluso il caso speciale di PrivateIpAddress.

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

Descrizione

needs-triage

Summary

`awscli/customizations/ec2/runinstances.py`'s `_fix_args` (lines ~120-141) moves several scalar top-level params (`SubnetId`, `SecurityGroupIds`, `Ipv6AddressCount`, `Ipv6Addresses`, `EnablePrimaryIpv6`) into the `NetworkInterfaces[0]` structure using six nearly-identical `if key in params: interface[dest] = params[key]; del params[key]` blocks.

Only the source/dest key names differ between blocks (aside from the `PrivateIpAddress` case, which needs real transformation into a `PrivateIpAddresses` list). This is pure copy-paste boilerplate that has grown over time as EC2 added more network-interface-eligible params, and is easy to get wrong (typo in source vs dest vs delete key) when extended further.

Proposed change

Replace the simple rename-and-move blocks with a declarative mapping (e.g. a `SIMPLE_PARAM_MAP` dict of source key -> destination key) and a small loop, keeping the `PrivateIpAddress` case as the one special-cased transform. This shortens the function and makes future additions a one-line map entry instead of a new if-block.

No behavior change intended — this is a pure refactor.

Lingua principale: Python
Stelle: 17.3k
Fork: 4.7k
Merge medio: 1g 7h
PR unite (30g): 21

Guida per i contributori

Apri la guida per i contributori

Come iniziare

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