Ideas for things to change/remove in the next major version (2.0)

Aperta
#114 15 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
25/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
php
Ambito
compilers

Direzione di ricerca

Esamina l’uso di AST_LIST in src/Phan/Analysis/PreOrderAnalysisVisitor.php e gli entry point ast\parse_code, ast\parse_file e reflection arginfo menzionati nell’issue. Controlla i riferimenti del README ai flag legacy, quindi definisci il completamento come la rimozione degli AST_LIST e dei flag deprecati elencati e la rappresentazione coerente dell’argomento di versione obbligatorio.

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

Descrizione

  1. Remove AST_LIST and mark it as deprecated, and remove it whenever AST version 50 is removed.

Related to #94 - It was overlooked

AST_LIST is still used in https://github.com/phan/phan/blob/1.2.2/src/Phan/Analysis/PreOrderAnalysisVisitor.php#L613 (incorrectly, will fix)

  1. Change the reflection arginfo of ast\parse_code and ast\parse_file to make the int $version mandatory

Remove legacy flags such as ast\flags\RETURNS_REF (alias of ast\flags\FUNC_RETURNS_REF mentioned in README)

Lingua principale
PHP
Stelle
983
Fork
81
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

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 nikic/php-ast

Tutte le issue di nikic/php-ast

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.