Skip to content

Upgrade IntelliJ Platform Gradle Plugin to 2.0 #4524

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

Merged
merged 97 commits into from
Jul 1, 2024
Merged

Conversation

rli
Copy link
Contributor

@rli rli commented May 29, 2024

Required for 2024.2 support

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@@ -32,7 +32,7 @@ allprojects {
configurations.all {
resolutionStrategy {
failOnDynamicVersions()
failOnChangingVersions()
// failOnChangingVersions()
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: do we need this or can be deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ideally we keep this, not sure if we can yet

rli and others added 16 commits June 25, 2024 17:46
* Improving telemetry error messages

* Adding reasonDesc param to telemetry error messages
Rewrite integration with the New Solution dialog in Rider to use the new API. Without this, in 2024.1 the plugin won't provide any extension for the New Solution dialog.

Co-authored-by: Richard Li <742829+rli@users.noreply.github.com>
Sometimes we get exactly 5 failing flaky tests
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
7.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

- su codebuild-user -c "./gradlew -PideProfileName=$ALTERNATIVE_IDE_PROFILE_NAME check coverageReport --info --console plain --continue"
- ./gradlew -PideProfileName=$ALTERNATIVE_IDE_PROFILE_NAME :plugin-toolkit:intellij-standalone:buildPlugin :plugin-core:buildPlugin :plugin-amazonq:buildPlugin
- su codebuild-user -c "./gradlew -PideProfileName=$ALTERNATIVE_IDE_PROFILE_NAME check coverageReport --info --console plain --continue -scan"
- ./gradlew -PideProfileName=$ALTERNATIVE_IDE_PROFILE_NAME :plugin-toolkit:intellij-standalone:buildPlugin :plugin-core:buildPlugin :plugin-amazonq:buildPlugin :plugin-toolkit:intellij-standalone:verifyPlugin :plugin-core:verifyPlugin :plugin-amazonq:verifyPlugin
Copy link
Contributor

Choose a reason for hiding this comment

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

is verifyPlugin running more tests than check?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the new behavior appears to spin up multiple IDE instances and we run out of RAM in the build container

.executeInBackground(
object : ProcessListener {
override fun startNotified(event: ProcessEvent) {
println("ssh client: startNotified")
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these print statements required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

trying to understand the random failure in windows for this test

@rli rli changed the base branch from main to feature/242 July 1, 2024 20:30
@rli rli merged commit 0d7d825 into feature/242 Jul 1, 2024
9 of 16 checks passed
@rli rli deleted the rli/intellij-gradle-2 branch July 1, 2024 20:30
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.

4 participants