prestodb/presto

Exclude module-info.class from byte code checks

Open

#22 193 ouverte le 13 mars 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Java (5 240 forks)batch import
buildcleanupgood first issue

Métriques du dépôt

Stars
 (15 558 stars)
Métriques de merge PR
 (Merge moyen 34j 14h) (120 PRs mergées en 30 j)

Description

The build is full of these warnings:

[WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53

This is not a surprise. module-info.class is something for java 9+ (byte code 53) while most of the build is Java 8 (byte code version 52).

This isn't a big deal since Java 8 will ignore module-info.class. There's no real problem here, but if anyone happens to know a straight-forward way to suppress these particular warnings that would be nice.

Guide contributeur

Direction de recherche
Examinez la configuration de build (probablement Maven) pour trouver où les vérifications de version du bytecode sont effectuées. Déterminez comment exclure module info.class de ces vérifications, éventuellement en configurant le plugin maven bundle plugin ou animal sniffer, ou en vous assurant que la build respecte la version de la classe module info.
Stack technique
java
Domaine
backenddatabase
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaMaven
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65