Add JPMS module-info.java
- Lingua principale
- Kotlin
- Stelle
- 101
- Fork
- 18
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
I'm not sure that anyone will ever need this. But I mistakenly thought that we did to reproduce a user bug, so I started experimenting in the `feat/module-info` branch ([see diff](https://github.com/diffplug/selfie/compare/main...feat/module-info)).
The hiccups I ran into were:
- the `module-info.java` has to be in a `java` folder, not a `kotlin` folder
- stuff that is in the `java` folder can't reference stuff in the `kotlin` folder
- we needed fake package-private `NeededForModuleInfo.java` files in the java folder to avoid compilation errors in the `module-info.java`
If you need selfie to support `module-info.java`, I'm happy to merge a PR that does it, and you can use this as a starting point.
- the hiccups above are fine with me. If you can fix the workarounds great, but I don't mind shipping them
- please merge in from main first, maybe the latest version of the Kotlin plugin will have made it easier by then
- I would like the metadata (naming and whatsuch) to be canonical, and I don't have enough experience with JPMS to know what it should be. Hopefully if you need JPMS, you've also got enough experience that you can set this metadata better than I did.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia esaminando il diff del branch feat/module-info e facendo il merge dell'ultimo main, quindi ispeziona il suo module-info.java e i file fittizi NeededForModuleInfo.java nella cartella java. Conferma i metadati canonici di JPMS con i manutentori del progetto o nella documentazione di JPMS. Il lavoro è completato quando selfie supporta module-info.java e i file richiesti vengono compilati correttamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java, kotlin
- Ambito
- build-system
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100