diffplug / diffplug/spotless

Investigate Javenode and/or JNpm

オープン
#934 コメント 0 件 リアクション 4 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Java
スター
5.6k
フォーク
559
平均マージ
1日 14時間
マージ済み PR(30日)
43

説明

Right now, we integrate with node.js formatting tasks by finding a Node.js and NPM installation on the host machine

- using npm to install the required formatter, usually prettier, which is slow (#766)
- [JNPM](https://github.com/OrienteerBAP/JNPM) might be able to do this better
- starting a localhost server in java, and another in node.js, and passing strings back and forth between them
- [Javenode](https://github.com/caoccao/Javenode) would allow us to do this in-process.
- Upside: fancy [module intercepting](https://github.com/diffplug/spotless/issues/556#issuecomment-919613345) which could cooperate with something like JNPM
- Upside: passing strings within-process would be faster, but this is minor
- Upside: don't need to futz with Spotless finding a node.js installation
- Downside: can't futz with Spotless using a specific node.js installation which might match other non-java tooling

I don't plan to work on any of this, but I think #766 is a big problem. If anyone wants to experiment with the above, they are very welcome to! An important part for getting a PR merged will be a real-world project, where you can say "it was X seconds before, and it's Y seconds now". I don't want to swap things just to swap them.

If there's an easy way to keep the old mechanism and the new mechanism at the same time, that might be nice, but it's fine to remove the old mechanism as part of the PR too. Maybe the new mechanism dominates the old, and if it doesn't we can figure out how to get them to coexist after the fact.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

#766 とリンクされている Javenode および JNPM プロジェクトを調査し、現在の Node.js formatter のワークフローと考えられる代替案を理解してください。完全な調査には、既存のメカニズムと提案されたメカニズムの実環境での比較を含め、変更前後の所要時間を報告し、古いアプローチと新しいアプローチを併存させるべきかどうかを説明する必要があります。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
java, node.js
領域
build-system, tooling
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。