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

PRの変更をオンラインでレビューできる仕組みが欲しい #2053

Closed
pocke opened this issue Dec 16, 2019 · 3 comments
Closed

PRの変更をオンラインでレビューできる仕組みが欲しい #2053

pocke opened this issue Dec 16, 2019 · 3 comments

Comments

@pocke
Copy link
Member

@pocke pocke commented Dec 16, 2019

ほしい物

PRをレビューするさいに実際に変更されたHTMLを見るには手元でfetchしてビルドする必要があってちょっと大変なので、オンラインでそれを確認できるものがほしい

解決策

Netlifyを使うと手軽に実現できそう

https://twitter.com/zundan/status/1206396995058855936

他の案

  • Heroku Review Apps
    • 課金しないとつらそう
    • 動的Webページがメインターゲットなのでオーバースペックかも
  • CircleCI Artifact
    • 使ったことがないからわからない…
  • GitHub Actions
    • 仕組みはありそうだけど使ったことがないからわからない

不安要素

  • Netlifyでホストされるページでは検索避けをしないと、本番の docs.ruby-lang.org の検索順位が下がってしまいそう?
@hanachin

This comment has been minimized.

Copy link
Member

@hanachin hanachin commented Dec 19, 2019

doctreeのレビューもそうですがbitclustの方でHTMLのデータ構造を変えたときもオンラインレビューできると確認が楽そうなので、bitclustの方にもほしいですね

@pocke

This comment has been minimized.

Copy link
Member Author

@pocke pocke commented Dec 24, 2019

resolved by #2062

@pocke pocke closed this Dec 24, 2019
@znz

This comment has been minimized.

Copy link
Member

@znz znz commented Dec 24, 2019

  • meta-robots-content のデフォルトが noindex なので、ロボット避けは大丈夫そうです。
  • --canonical-base-url も localhost をさしておく意味はないので、ローカル生成も含めて Rakefile でのデフォルトを変更しました。
  • 以前試した時に netlify は /tmp 以下のファイルの publish がうまくいかなかった覚えがあるので、 HTML_DIRECTORY_BASE を指定できるようにしました。
  • 以前試した時に netlify で複数バージョン生成すると時間切れで失敗していた覚えがあるので、 2.6.0 だけにしています。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.