eclipse-jdt/eclipse.jdt.core

Follow up on minimal target change to 1.8

Open

#2761 aperta il 25 lug 2024

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)Java (183 fork)auto 404
compilerhelp wanted

Metriche repository

Star
 (233 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Open TODO's from https://github.com/eclipse-jdt/eclipse.jdt.core/pull/2551

Created issues:

Following questions wait for answering by others:

  • [Stephan] Should this.currentProject.setOption(JavaCore.COMPILER_PB_RAW_TYPE_REFERENCE, JavaCore.IGNORE); be used in ConverterTestSetup.setUpJavaProject(String, String) ?
  • [Stephan]: change on BatchCompilerTest.testBridgeMethodRetention This change probably conflicts with the purpose of this test, @srikanth-sankaran ?
  • [Stephan] Change in ASTConverterTestAST8_2.test0612() caught me by surprise so I investigated:
    • the test implies to test AST at JLS8
    • using source from project Converter, Converter15, however, selects the AST level from the project. @srikanth-sankaran , @jarthana does it make any sense have such tests here? Shouldn't tests always sit in a test class corresponding to the used Converter* project, e.g., ASTConverterAST3Test?

Guida contributor

Direzione di ricerca
Esamina la PR #2551 e i problemi correlati per comprendere il contesto della modifica del target minimo a 1.8. Affronta le domande aperte di Stephan riguardanti la configurazione dei test e il posizionamento dei test. Considera l'implementazione del problema rimanente non spuntato #2757.
Tech stack
java
Dominio
backend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70