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

Network.HTTP: A new HTTP module [WIP] #398

Closed
wants to merge 13 commits into from
Closed

Conversation

lambdalisue
Copy link
Member

@lambdalisue lambdalisue commented Feb 14, 2016

とりあえず雛形がないと作りにくいと思ったので Python バージョンを作ってみました。
Async に関してはまだ対応していません。
#397 が終わったら curl / wget 版も作って既存の Web.HTTP を置き換えましょう!

@lambdalisue
Copy link
Member Author

vital.vim って各プラグインごとだから今の Python の import の仕組みだと

  • PYTHON_PATH がエグいことになる
  • 別バージョンがあった場合にすでにロード済みだと再ロードされないからAPI変わると死ぬ

の二点からつらそう。関数型名前空間のほうに変えるか…関数型名前空間なら衝突しても上書きだし。
ただそうするとテストが面倒だけど仕方ないか……

@lambdalisue
Copy link
Member Author

メモ: こいつでも #397 (comment) みたいに Mock 対応して開発を加速する

@thinca
Copy link
Member

thinca commented Aug 7, 2016

これって今どういうステータスですっけ。

@lambdalisue
Copy link
Member Author

ステータス:放置

って感じですかね Mock 対応版作製暇を見つけて行います。 wget/curl 版は別 PR がいいかなと思うのですがどうでしょう?(この PR では全体の仕様を決める)

@raa0121
Copy link
Member

raa0121 commented Jan 8, 2017

wget/curl 版は別 PR がいいかなと思うのですがどうでしょう?(この PR では全体の仕様を決める)

私は別 PR で良いと思います。
時間が空いてしまったからか、マージするにはコンフリクトを解消しないといけない状態になってしまってますね…。

@ujihisa
Copy link
Member

ujihisa commented Jun 13, 2018

I'm checking through all existing pull requests. Should I leave this pull request open to let people discuss more, or close this?

@lambdalisue
Copy link
Member Author

Let's close

@ujihisa ujihisa deleted the feature/Network.HTTP branch June 13, 2018 11:49
@ujihisa ujihisa restored the feature/Network.HTTP branch June 13, 2018 11:49
@ujihisa ujihisa deleted the feature/Network.HTTP branch June 30, 2019 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants