Skip to content

New Version of BlueskyGephi#332

Open
totetmatt wants to merge 7 commits into
gephi:master-forgefrom
totetmatt:blueskyGephi
Open

New Version of BlueskyGephi#332
totetmatt wants to merge 7 commits into
gephi:master-forgefrom
totetmatt:blueskyGephi

Conversation

@totetmatt

Copy link
Copy Markdown

New plugin or plugin update?

  • [] New Plugin
  • Update

What is the purpose of this plugin?

  • Making gephi 0.11.1 compatible
  • Can connect to different host (like eurosky.social)

How to test your plugin in Gephi?

Checklist before submission

  • Did you merged with the master branch to get the latest updates?
  • Did you build and test the plugin successfully locally?
  • Did you include metadata (e.g. plugin author, license) in the pom.xml file
  • Did you make sure NOT to include any unnecessary files in your PR?
  • Did you write unit tests to test your plugin functionalities?

@totetmatt totetmatt requested review from eduramiba and mbastian June 30, 2026 17:05

@eduramiba eduramiba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me!

Unrelated: now it's necessary to compile with java 17 to avoid compilation issue in other plugin (error in modules/Multimode-Networks/src/main/java/cz/cvut/fit/gephi/multimode/MultiModeWindowTopComponent.java).

Also, we should update junit it seems, or build with skipTests, otherwise you will get:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.6.0-M1:test (default-test) on project streaming-impl: The used JUnit Version 4.10 is not supported anymore. Please update to version 4.12+

But I think code in this plugin to be merged is fine, and the build process of master-forge will also be fine because it uses java 17 and does not seem to run tests.

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.

2 participants