search

getappmap / appmap-intellij-plugin

MIT License
38 stars 9 forks source link

Drop code which was only used by <=2023.1 #683

Closed jansorg closed 7 months ago

jansorg commented 7 months ago

AppMapIndexableFilesContributor was only sed with 2023.1 and earlier versions, but we've dropped compatibility with 2023.1 and earlier.

AppMapIndexedRootsSetContributor is the implementation in use with 2023.2 and later.

github-actions[bot] commented 7 months ago

AppMap runtime code review

Summary Status
Failed tests :white_check_mark: All tests passed
API changes :zero: No API changes
Security flaws :white_check_mark: None detected
Performance problems :white_check_mark: None detected
Code anti-patterns :white_check_mark: None detected
New AppMaps :star: 2 new junit tests
Removed AppMaps :heavy_multiplication_x: 2 removed junit tests

⭐ New AppMaps

[junit] App map file lookup scoped locate filename by index from plugin-core/src/test/java/appland/files/AppMapFileLookupTest.java:36

[junit] App map indexed roots set contributor index from plugin-core/src/test/java/appland/index/AppMapIndexedRootsSetContributorTest.java:14

✖️ Removed AppMaps

[junit] App map indexable files contributor files

[junit] App map indexable files contributor index

appland-release commented 7 months ago

:tada: This PR is included in version 0.65.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: