Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(intellij-plugin): makes it compatible with jetbrains app version… #436

Conversation

stanlee974
Copy link
Collaborator

@stanlee974 stanlee974 commented Jan 18, 2024

Update the intellij plugin version to make it compatible with Intellij version 233 +

Reopen of #434

Copy link
Collaborator Author

@stanlee974 stanlee974 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cette portion de code porte sur la version d'intelliJ utilisée pour lancer les tests du plugin. Il va lancer cet intelliJ avec le plugin installé. C'est effectivement intéressant de la monter.

En montant cette version, on doit monter la version du plugin gherkin à la version 233.11799.165 pour qu'il soit compliant avec 233.*

On peut tenter une correction en mettant ça dans patchPluginXml {
updateSinceUntilBuild.set(true)
sinceBuild.set("222")
untilBuild.set("")
}

comme dans cette exemple https://github.com/JetBrains/ideolog/blob/5365af1ee1d3fcfa7404bffe4fa0311d4975eb34/build.gradle.kts#L37

Si ça ne fonctionne pas, on peut faire :
patchPluginXml {
updateSinceUntilBuild.set(true)
sinceBuild.set("222")
untilBuild.set("233.*")
}

mais il faudra le changer souvent :-(

@stanlee974 stanlee974 requested a review from luifr10 January 21, 2024 15:28
@stanlee974
Copy link
Collaborator Author

stanlee974 commented Jan 21, 2024

J'ai push une branche de correction qui fonctionne chez moi. @luifr10 je te laisse revérifier chez toi.
Si c'est ok, une fois packagé, @geromegrignon je te laisserai revérifier pour voir si c'est mieux.

@geromegrignon
Copy link
Contributor

Ok je testerai cela.
Ce serait intéressant de creuser sur la façon de gérer cela pour d'autres plugins.
Je ne sais pas s'il y a des webhooks qui permettraient de pouvoir écouter des changements de versions à brancher avec une GitHub Action pour faire un commit automatique par exemple.

@stanlee974
Copy link
Collaborator Author

stanlee974 commented Jan 23, 2024

On doit encore régler quelque petites choses. On s'est rendu compte que la modif qu'on a faite, rend la dernière version non compliant avec la 222.* (je vais créer un bug pour tracer le soucis)
Ton idée est intéressante. On aura quand même une petite logique, parce que comme on utilise Gherkin également, il faut garantir que sa version soit compatible avec celle de intelliJ également.
Je l'ai tracé en issue https://github.com/Orange-OpenSource/uuv/issues/452

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants