import を整理することについて about .github HOT 17 CLOSED

この2択しかないのなら個人的には以下の方が良いと思います。

from geometry_msgs.msg import Transform
from geometry_msgs.msg import TransformStamped

今のところ、上記の書き方以外isortを使う異議が無いのでまた何かあれば別のissueを立てます。
もし大丈夫なら--force-single-line-imports追加して欲しいです。

多分調べ済みだと思いますが一応isortのissueにもありました。

from .github.

isortのconfigでは同一パス内のものをimportしているときにgroupingを強制しない設定はなかったです。
flake8-import-orderではその辺りはゆるくもう少し自由に書けます。
例

よって、

1. isortのsortingに慣れる
2. flake8-import-orderにしてwarning出ているところは手動でsortingする
   のどちらかになります。

from .github.

ミーティングでも言いましたし、

手動でsortingする

と書いてあるようにないです。
force-single-line-importsは1行に1importしかしないように強制しますよね？
joshuaの例だと
from geometry_msgs.msg import Transform, TransformStamped
も分解する。

isortでは改行が必要になったときのstyleがいくつか選べますが、max-lengthの119を超えない限りは1行に収めようとします。
https://pycqa.github.io/isort/docs/configuration/multi_line_output_modes.html

またisort:skipは明確な判断基準が作りにくく全てにisort:skipされても止めようがないのでその方法は却下です。

from .github.

そこでjoshuaが折れたらじゃあisortこのまま使おうかになるだけなんだけど...

from .github.

@sbgisen/developer 自分はflake8-import-orderのほうが強制しすぎないので好きなので変えるのは賛成ですが、皆さん大丈夫ですか？

from .github.

行が長くなりすぎるのを避けたい場合もあるのでできれば変えてほしいです。

from .github.

2.のflake8-import-orderにしてもいいような気がしますが、isortコマンドのようなFormatterってありますかね？

※Option外なので少しキモいですが、isort:skip というのもあるようです。
https://pypi.org/project/isort/

from xyz import (abc,  # isort:skip
                 yo,
                 hey)

from .github.

2.のflake8-import-orderにしてもいいような気がしますが、isortコマンドのようなFormatterってありますかね？

自己解決しました。isortに渡すときに、 -sl, --force-single-line-imports のオプション渡せばいいようです。

from .github.

結局isortを使う場合

from geometry_msgs.msg import Transform, TransformStamped

from geometry_msgs.msg import Transform
from geometry_msgs.msg import TransformStamped

どちらか一つしか選べないのならどちらに統一するのかな？

from .github.

force-single-line-importsだと今回の要件が満たされないのは理解しました。

flake8-import-orderのisortコマンドのようなFormatterってありますかね？

こちらはいかがですかね？
formatterがないのであれば、並べ替えの手間が発生するのでisrotのままが良いです。

from .github.

ミーティングでも言いましたし、
手動でsortingする
と書いてあるようにないです。

??????

from .github.

ミーティングでも言いましたし、
手動でsortingする
と書いてあるようにないです。

??????

すみません。主語がなくて分かりませんでした。無いんですね。

from .github.

ミーティングでも言いましたし、
手動でsortingする
と書いてあるようにないです。

??????

すみません。主語がなくて分かりませんでした。無いんですね。

引用忘れてました。
いちいち全てに引用つけないと前後のスレッド見れませんか失礼いたしました以後気をつけます

from .github.

いちいち全てに引用つけないと前後のスレッド見れませんか失礼いたしました以後気をつけます

こちらこそ、スレッドちゃんと読めていませんでした。すみません。
というより、理解が出来ていなかったです。理解が遅くて申し訳ないです。

from .github.

1. isortのsortingに慣れる
2. flake8-import-orderにしてwarning出ているところは手動でsortingする

これであれば、1が良いです。

from .github.

@jsupratman13

PR #49 でマージした結果以下の書き方ができなくなりました。個人的に前の書き方の方が見やすいと思いますので自動整理不可能であればisort使わずにこちらで整理した方がいいと思いますががいかがでしょうか？

どのような書き方が「見やすい」かは個人差あると思うのですが、私としては「統一されている方が見やすい」というのは個人差ないと思っています。なのでフォーマットをかけること自体は必要だと思っています。

結局isortを使う場合

from geometry_msgs.msg import Transform, TransformStamped

from geometry_msgs.msg import Transform
from geometry_msgs.msg import TransformStamped

どちらか一つしか選べないのならどちらに統一するのかな？

この2択だとジョシュアさんはどちらにしたいのですか？

from .github.

一旦issueを閉じます。

from .github.