-
-
Notifications
You must be signed in to change notification settings - Fork 3
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: 2.7.0 #141
base: master
Are you sure you want to change the base?
feat: 2.7.0 #141
Conversation
fix: #133 Support IntelliJ IDEA 2024.3 (build 243)
Noting for the record the following errors and warnings occurred with
That said, the build still "succeeded" |
Gradle Wrapper version needs to be updated to 8.11.1
|
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments on code
@@ -8,7 +8,7 @@ plugins { | |||
} | |||
|
|||
group = "com.asyncapi.plugin.idea" | |||
version = "2.6.0+jre17" | |||
version = "2.7.0+jre17" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be
version = "2.7.0+jre21"
Also a few lines up: org.jetbrains.intellij.platform version 2.2.0 released Dec 6. The master
branch has this...why isn't it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+jre17 is a valid version of jdk
I fixed typos in CHANGELOG.md
## 2.7.0+jre21 | ||
published to: | ||
- [JetBrains marketplace](https://plugins.jetbrains.com/plugin/15673-asyncapi/versions/stable/643494) | ||
- [GitHub](https://github.com/asyncapi/jasyncapi-idea-plugin/releases/new?tag=2.7.0%2Bjre17) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tag should reference
jre21
NOT
jre17
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. jre17 is valid tag
"2024.2.2", | ||
"2024.2.3", | ||
"2024.2.4", | ||
"2024.2.5" | ||
)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about 2024.3 and 2024.3.1?
It won't let me comment later in the file:
As discussed above, this should be based upon JDK 21:
- Please update both
jvmTarget
to21
- Please update
JavaLanguageVersion
to21
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2024.2.5 is a last version of IDEA for v2
2024.3 is supporting by v3 - https://github.com/asyncapi/jasyncapi-idea-plugin/tree/release/3.0.0
|
|
Quality Gate passedIssues Measures |
Added
Fixed