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

Release new version of JetBrains plugin #559

Merged
merged 21 commits into from
Feb 5, 2024
Merged

Release new version of JetBrains plugin #559

merged 21 commits into from
Feb 5, 2024

Conversation

ahtrotta
Copy link
Contributor

@ahtrotta ahtrotta commented Feb 5, 2024

No description provided.

This helps during development to quickly locate errors or warnings in our webviews. It's disabled by default.
@ahtrotta ahtrotta self-assigned this Feb 5, 2024
Copy link

github-actions bot commented Feb 5, 2024

AppMap runtime code review

Summary Status
Failed tests ✅ All tests passed
API changes 🚧 1 breaking
Security flaws ✅ None detected
Performance problems ✅ None detected
Code anti-patterns ✅ None detected
New AppMaps ⭐ 5 new junit tests
Removed AppMaps ✖️ 1 removed junit test

🔄 API changes

🚧 Breaking changes

These changes are not backwards-compatible, according to the OpenAPI specification.

  • Remove path /api/appmaps/create_upload
Detailed OpenAPI diff
--- base/openapi.yml	2024-01-14 00:11:59.000000000 +0000
+++ head/openapi.yml	2024-02-05 19:44:33.000000000 +0000
@@ -15,12 +15,6 @@
 info:
   title: My project
   version: v1
-paths:
-  /api/appmaps/create_upload:
-    post:
-      responses:
-        '200':
-          content: {}
-          description: OK
+paths: {}
 components: {}

⭐ New AppMaps

[junit] Default command line service indexer json rpc port from plugin-core/src/test/java/appland/cli/DefaultCommandLineServiceTest.java:210

[junit] Open external links handler is external url from plugin-core/src/test/java/appland/webviews/OpenExternalLinksHandlerTest.java:13

[junit] App map webview base url from plugin-core/src/test/java/appland/webviews/webserver/AppMapWebviewTest.java:19

[junit] App map webview web view index html urls from plugin-core/src/test/java/appland/webviews/webserver/AppMapWebviewTest.java:34

[junit] App map webview webview url from plugin-core/src/test/java/appland/webviews/webserver/AppMapWebviewTest.java:26

✖️ Removed AppMaps

[junit] App map uploader app map upload

@ahtrotta ahtrotta marked this pull request as ready for review February 5, 2024 19:53
@dustinbyrne dustinbyrne self-requested a review February 5, 2024 19:54
@dustinbyrne dustinbyrne merged commit 344e776 into main Feb 5, 2024
14 checks passed
@appland-release
Copy link
Contributor

🎉 This PR is included in version 0.55.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants