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

Build for idea version 2021.3. Fixed #391 #392

Merged
merged 7 commits into from Dec 5, 2021

Conversation

fishermans
Copy link
Contributor

Please find my fixes for #391. Hopefully, it fixes all issues.

@sfragata
Copy link

sfragata commented Dec 2, 2021

LGTM
@dubreuia what you think ?

@fishermans
Copy link
Contributor Author

This commit seems also to be valid for 2021.2.3. I used it the whole day without any issues neither on 2021.3 nor on 2021.2.3.
If desired I can downgrade the min version to 212. Please let me know.

@vulinh64
Copy link

vulinh64 commented Dec 3, 2021

plz implement this. the stupid action on save doesn't help a lot :(

import javax.swing.JComboBox;
import javax.swing.JComponent;
import javax.swing.JPanel;
import javax.swing.*;
Copy link

Choose a reason for hiding this comment

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

I think you should keep these imports instead of use *

import javax.swing.event.ListDataEvent;
import javax.swing.event.ListDataListener;
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.*;
Copy link

Choose a reason for hiding this comment

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

I think you should keep these imports instead of use *

import javax.swing.JPanel;
import java.awt.BorderLayout;
import java.awt.Dimension;
import javax.swing.*;
Copy link

Choose a reason for hiding this comment

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

I think you should keep these imports instead of use *

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree. I used the code style attached to this project.

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be fixed by the last commit.

@sfragata
Copy link

sfragata commented Dec 3, 2021

Now I think we need to wait @dubreuia blessing :)

@vulinh64
Copy link

vulinh64 commented Dec 5, 2021

Now I think we need to wait @dubreuia blessing :)

Did he go MIA somewhere?

@fishermans
Copy link
Contributor Author

Now I think we need to wait @dubreuia blessing :)

Did he go MIA somewhere?

Meanwhile, you can build the plugin yourself locally and install it locally. Choose the gear under Plugin and select install from disk.

@dubreuia
Copy link
Owner

dubreuia commented Dec 5, 2021

Looking, thanks for the work

@fishermans
Copy link
Contributor Author

Looking, thanks for the work

Recently, I spent more time in removing all pending deprecated API calls and made the Plugin reloadable. Let me know if I should update this PR.

@dubreuia dubreuia merged commit 4196f87 into dubreuia:main Dec 5, 2021
@dubreuia
Copy link
Owner

dubreuia commented Dec 5, 2021

JetBrains is reviewing this update. Please expect it to be approved within two business days of the upload.

@dubreuia
Copy link
Owner

dubreuia commented Dec 5, 2021

@sfragata @fishermans Thank you so much for your work on the plugin, I really appreciate it 🙏

I've uploaded plugin version 2.3.0 to Jetbrains plugin repo, should be up by Monday. I've tested the plugin on my side, the changes looks ok to me.

Recently, I spent more time in removing all pending deprecated API calls and made the Plugin reloadable. Let me know if I should update this PR.

@fishermans That would be great if you don't mind preparing another PR? Sorry I already merged & released this one, I wanted to push the new version ASAP. Still good to cleanup & improve

@fishermans
Copy link
Contributor Author

@sfragata @fishermans Thank you so much for your work on the plugin, I really appreciate it 🙏

I've uploaded plugin version 2.3.0 to Jetbrains plugin repo, should be up by Monday. I've tested the plugin on my side, the changes looks ok to me.

Recently, I spent more time in removing all pending deprecated API calls and made the Plugin reloadable. Let me know if I should update this PR.

@fishermans That would be great if you don't mind preparing another PR? Sorry I already merged & released this one, I wanted to push the new version ASAP. Still good to cleanup & improve

You are welcome! I am using your Plugin each day. Thus, I have to say THANK YOU!
I will open a new PR for further improvements next week.

@fishermans fishermans deleted the idea-version-2021.3 branch December 5, 2021 21:54
@sfragata
Copy link

sfragata commented Dec 6, 2021

@dubreuia all cheers to @fishermans , He did the fix, I did the review ;)

Thank you for the great plugin, my team uses Eclipse and Intelj so I need to use the same formatter/organizer, so your plugin is perfect for my needs

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.

None yet

4 participants