Hi,

I tried to configure it in a new computer without code style file.
Unfortunately, it merged java packages with *.

Then I imported the code style file again, and again, the order of import
file was incorrect.

Thanks the help of @Steve Su <[email protected]>  I found the solution
of this problem.

Search code style in preferences -> choose Java -> choose Imports -> choose
Imports Layout below
-> Then modify the order of imports like:
import all other imports
<blank line>
import java.*
<blank line>
import static all other imports

I hope it helps if you encounter the same problem.

jincheng sun <[email protected]> 于2021年2月24日周三 上午9:38写道:

> Thanks for the feedback XiangWei!
>
> Just like Xiangdong said, we do not need download the code style file just
> import from our project is enough.
> I'd like to know if the plug-in works well to you now?
>
> Best,
> Jincheng Sun 孙金城
>
>
> Xiangdong Huang <[email protected]> 于2021年2月23日周二 上午12:03写道:
>
> > > The import ordering is not handled by this plugin
> >
> > Really? On my IDEA, it works.
> > I did not set the step: "download the IntelliJ Java Google Style file and
> > import it into File→Settings→Editor→Code Style"
> >
> > Best,
> > -----------------------------------
> > Xiangdong Huang
> > School of Software, Tsinghua University
> >
> >  黄向东
> > 清华大学 软件学院
> >
> >
> > Xiangwei Wei <[email protected]> 于2021年2月22日周一 下午4:13写道:
> >
> > > Hi Jincheng,
> > >
> > >
> > > I followed the user guide in [1], installed the google-java-format
> plugin
> > > and Save Actions plugin. But it seems the import ordering is not
> handled
> > > properly.
> > > In the site of google-java-format [2], the problem is mentioned:
> > >
> > > "The import ordering is not handled by this plugin, unfortunately.
> > > To fix the import order, download the IntelliJ Java Google Style file
> and
> > > import it into File→Settings→Editor→Code Style."
> > >
> > > I did like this, but it was not working. Do you have some solutions?
> > >
> > > Thank you. Regards
> > >
> > >
> > > [1]
> > >
> http://iotdb.apache.org/Development/ContributeGuide.html#code-formatting
> > > [2]
> > >
> > >
> >
> https://github.com/google/google-java-format#intellij-android-studio-and-other-jetbrains-ides
> > >
> > >
> > > jincheng sun <[email protected]> 于2021年2月19日周五 上午9:46写道:
> > >
> > > > It seems that step 7 should be performed first to ensure that the
> > commit
> > > > for step 2 is there.
> > > >
> > > > jincheng sun <[email protected]> 于2021年2月18日周四 下午8:57写道:
> > > >
> > > > > Hi folks,
> > > > >
> > > > > We have merged the PRs of IOTDB-1156, i.e., the Spotless with
> > > > > google-java-format now in effect.
> > > > >
> > > > > This change touched most of single java file in the
> repository(70K+).
> > > So
> > > > > most of PRs will require a rebase, follow the steps below:
> > > > >
> > > > > 1. git checkout YOUR_PR
> > > > > 2. git cherry-pick b1261ecb5893ee0d357fec311b4d92ff9039a989
> > > > > 3. mvn spotless:check
> > > > > 4. mvn spotless:apply
> > > > > 5. git add .
> > > > > 6. git commit -m 'Apply Spotless'
> > > > >
> > > > > # asf  [email protected]:apache/iotdb.git (fetch)
> > > > > # Pull the latest master branch
> > > > >
> > > > > 7. git checkout -b asf_master asf/master
> > > > > 8. git checkout YOUR_PR
> > > > > 9. git rebase asf_master
> > > > >
> > > > > # Resolve your conflict:
> > > > > # NOTE: `Accept Yours` for your PR involves files, and `accept
> > Theirs`
> > > > for
> > > > > other files.
> > > > >
> > > > > 10 git rebase --continue
> > > > > # Double check
> > > > > 11. mvn spotless:check
> > > > > # If you see "build success" successfully, it is proved that rebase
> > has
> > > > > been successful.
> > > > > 12. mvn clean package -DskipTests
> > > > > # If you see "build success" successfully, you can push your
> changes
> > to
> > > > > you PR branch.
> > > > >
> > > > > Then you can squash the commits if you want, Good luck!
> > > > >
> > > > > Most of the changes were done by the commit(70fc76de8b5a032), which
> > you
> > > > > can exclude from git blame by configuring git as follows:
> > > > >
> > > > > git config blame.ignoreRevsFile .git-blame-ignore-revs
> > > > >
> > > > > You can setup the IDE to follow the new code style according [1]
> > > > >
> > > > > Finally, a big thank you to Xiangdong and Dawei for review and
> merge
> > > PR,
> > > > > and the people who involved in the early discussions.[2]
> > > > >
> > > > > Best,
> > > > > Jincheng 孙金城
> > > > >
> > > > > [1]
> > > > >
> > > >
> > >
> >
> https://iotdb.apache.org/Development/ContributeGuide.html#code-formatting
> > > > > [2]
> > > > >
> > > >
> > >
> >
> https://lists.apache.org/thread.html/rd07d66cc819bce546aced54d7f0b0ab4552a910a120f05fe900915dc%40%3Cdev.iotdb.apache.org%3E
> > > > >
> > > >
> > >
> > >
> > > --
> > > Best,
> > > Xiangwei Wei
> > >
> >
>


-- 
Best,
Xiangwei Wei

Reply via email to