Comments (17)
この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ではその辺りはゆるくもう少し自由に書けます。
例
よって、
- isortのsortingに慣れる
- 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.
- isortのsortingに慣れる
- flake8-import-orderにしてwarning出ているところは手動でsortingする
これであれば、1が良いです。
from .github.
PR #49 でマージした結果以下の書き方ができなくなりました。個人的に前の書き方の方が見やすいと思いますので自動整理不可能であればisort使わずにこちらで整理した方がいいと思いますががいかがでしょうか?
どのような書き方が「見やすい」かは個人差あると思うのですが、私としては「統一されている方が見やすい」というのは個人差ないと思っています。なのでフォーマットをかけること自体は必要だと思っています。
結局isortを使う場合
from geometry_msgs.msg import Transform, TransformStampedfrom geometry_msgs.msg import Transform from geometry_msgs.msg import TransformStampedどちらか一つしか選べないのならどちらに統一するのかな?
この2択だとジョシュアさんはどちらにしたいのですか?
from .github.
一旦issueを閉じます。
from .github.
Related Issues (20)
- flake8-annotationsの出力
- flake8の一部プラグインがpre-commit時にコミットを弾かない HOT 4
- C801が正常に機能していないので無効化する HOT 4
- build workflowが1時間でキャンセルされてしまう
- sphinx_rtd_themeのバージョン
- Noetic is left in ros2 test workflow
- generate multiversions document HOT 1
- CI中にisortが正常に動かない HOT 21
- flake8が正常にインストールされておらず、走っていないにも関わらず落ちていない HOT 1
- ros2のパッケージでsetup.cfgがconflictする HOT 5
- ros2 build/testのworkflowにlibfreenect周りを追加する
- ros2でclang-tidyを実行するときstd library周りでエラーが出る
- ros2のテスト結果を確認できていない
- pythonのlinterが3.8より後のバージョンに対応していない
- ドキュメント生成時のros/ros2の分岐ミス
- test時にサーバにあるレポジトリと.rosinstallに書かれているレポジトリのリモートが異なる時 HOT 1
- pythonのformatter, linterをruffで置き換える?
- catkin config --install option?
- rosdep installのtimeout時間が短い
- self hosted runnerの管理が面倒
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from .github.