ast icon indicating copy to clipboard operation
ast copied to clipboard

Add support for annotations to KlassIdentifier refs. Issue#95

Open birajpatel opened this issue 2 years ago • 6 comments

Fixes : https://github.com/kotlinx/ast/issues/95

birajpatel avatar Apr 18 '23 10:04 birajpatel

@drieks Please review 😃.

birajpatel avatar Apr 18 '23 11:04 birajpatel

Hi @birajpatel , thank you very very much! Your pull request looks very good! Sadly, there is a problem with the build system. To fix this, I want to move the build to gitlab actions. I already migrated https://github.com/drieks/antlr-kotlin in this way (this is a dependency used by kotlinx.ast). The release action is working fine: https://github.com/drieks/antlr-kotlin/actions/runs/4536723459/jobs/8409480096, but the publication to maven cental is not working (https://github.com/kotlinx/ast/issues/89)

When this is fixed, I can use almost the same action for kotlinx.ast to upload it to maven central. after this, the build should be stable and this pull request should be able to build.

I'm really very sorry that the build is broken and that's why the pull request can't be merged at the moment :(

drieks avatar Apr 18 '23 21:04 drieks

@drieks Oh, thats really sad, Actually we are using this library for one of our internal tools and merging this fix will unblock some pending things. (Because without my fix above the parser just fails to parse the file). Please try to help ASAP 😄 🙏 I saw many other folks are also awaiting the mavenCentral release. 🙏

birajpatel avatar Apr 19 '23 07:04 birajpatel

Hey @drieks! Any news on this? I'm also interested in this fix 👀

renatacmn avatar May 04 '23 10:05 renatacmn

Any update on this? It would be very good to have this fix merged @drieks

aaronfodor avatar May 23 '23 08:05 aaronfodor

Hi, I'm really sorry, I don't have much time at the moment. one possibility would be to temporarily expand the antlr-kotlin support. but I'm still not sure why publishing to maven central doesn't work. in the end not all jar files that were necessary were created. and I think we will run into the same problem with this library here, because it seems to be a multiplatform publishing issue.

drieks avatar May 23 '23 09:05 drieks