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

Add (real) support for IDEA build 201 #295

Merged
merged 3 commits into from
May 3, 2020
Merged

Add (real) support for IDEA build 201 #295

merged 3 commits into from
May 3, 2020

Conversation

fabmars
Copy link

@fabmars fabmars commented Apr 30, 2020

Following #288
Considering 2020 deprecations
image
and based on https://www.jetbrains.org/intellij/sdk/docs/basics/plugin_structure/plugin_components.html

Here is the support for the gitflow plugin on IDEA 201.

@fabmars
Copy link
Author

fabmars commented Apr 30, 2020

Should solve #291 #292 #293

@fabmars
Copy link
Author

fabmars commented Apr 30, 2020

and #276

@OpherV
Copy link
Owner

OpherV commented Apr 30, 2020

Brilliant! Thanks for the effort - I'll try and review over the weekend and hopefully get a version out for early next week 😄

@fabmars
Copy link
Author

fabmars commented Apr 30, 2020

The handleMerge() "logic" is really hacky but that's an issue with idea's apis, nothing we can do. I tried to improve it though. That's what took 80% of the time spent here.


@Nullable
@Override
public TaskDialogPanel getOpenTaskPanel(@NotNull Project project, @NotNull LocalTask task) {
Copy link
Owner

Choose a reason for hiding this comment

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

What's this change about?

Copy link
Owner

Choose a reason for hiding this comment

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

Oh it's one of the deprecations. Nevermind :)

@OpherV
Copy link
Owner

OpherV commented May 3, 2020

Getting this while testing this PR:

image

@OpherV
Copy link
Owner

OpherV commented May 3, 2020

Solved by refactoring some of GitflowVerisonTester and its init functionality

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