Skip to content

slack development 2020 03 16

akabeko edited this page Apr 19, 2020 · 1 revision

2020-03-16

Time (UTC) Icon Name Message
08:14 shinyu https://github.com/vivliostyle/vivliostyle/pull/630
このpull reqで、Vivliostyle ViewerのVivliostyleロゴを新しくしました。テスト:
https://vivliostyle-git-new-logo.vivliostyle.now.sh/


ところで、このようなコミットのConventional Commitsのメッセージはどうするのがよいのでしょう?
わからないので “update Vivliostyle logo images” とだけ書きました。
[vivliostyle/vivliostyle] #630 update Vivliostyle logo images
08:14 Kiara Translation 🇬🇧: https://github.com/vivliostyle/vivliostyle/pull/630
In this pull req, we updated the Vivliostyle logo of Vivliostyle Viewer. test:
https://vivliostyle-git-new-logo.vivliostyle.now.sh/


By the way, what about the messages of the Conventional Commits of such commits?
I just wrote “update Vivliostyle logo images” because I don't know.
08:31 uetchy feat: update logo imageのような感じで良いと思います
08:31 Kiara Translation 🇬🇧: feat: update logo image
08:33 uetchy ちなみにfeat!: やfix!:のように、種類ラベルの後に!を付けると破壊的変更という意味になり、次のリリースがメジャーリリースになります
08:33 Kiara Translation 🇬🇧: By the way, if you add! After the type label like feat !: or fix!:, It means a destructive change, and the next release will be a major release
09:15 shinyu なるほど、デザイン的な変更もfeatに含めるとよいのですね。
“feat: update logo image” にcommitメッセージを変更して、プルリクマージしました。
09:15 Kiara Translation 🇬🇧: Indeed, it is good to include design changes in feat.
Changed the commit message to “feat: update logo image” and made a pull recurrence.
09:56 shinyu プレリリース版Vivliostyle v2.0.0-pre.8 ができました。
https://github.com/vivliostyle/vivliostyle/releases/tag/v2.0.0-pre.8

正式リリース版 v2.0.0 はまだですが、リリース版のviewerの公開予定のURL https://vivliostyle.org/viewer/ で、テストできるように、現在のv2.0のviewerをこのURLで使えるようにできたらよいと思います。リニューアルWebサイトのサンプルページからリンクするViewerはこれになるので、新しいURLパラメータ仕様で動く必要があるので。

どうするとよいでしょう > @uetchy
09:56 Kiara Translation 🇬🇧: Pre-release version Vivliostyle v2.0.0-pre.8 is now available.
https://github.com/vivliostyle/vivliostyle/releases/tag/v2.0.0-pre.8

Although the official release version v2.0.0 is not yet released, the current version 2.0 viewer can be used at this URL so that you can test it at the URL https://vivliostyle.org/viewer/ that will release the release version viewer. I hope it can be used. The Viewer linked from the sample page of the renewal website will be this, so it needs to work with the new URL parameter specification.

What should I do?
09:59 uetchy yarn build 後/packages/viewer/lib以下にビルドしたアーティファクトが現れるのでそれをorgリポジトリにコピーしてあげれば使えます。こちらでやっておきます。
09:59 Kiara Translation 🇬🇧: After yarn build, built artifacts appear under / packages / viewer / lib. You can use them by copying them to the org repository. I'll do it here.
09:59 shinyu @uetchy Thanks!
09:59 Kiara Translation 🇯🇵:  ありがとう!
10:00 uetchy これはmasterに上げる?それとも2020ブランチで?
10:00 Kiara Translation 🇬🇧: Is this a master? Or on the 2020 branch?
10:01 shinyu 両方に必要だと思います。
10:01 Kiara Translation 🇬🇧: I think it is necessary for both.
10:02 uetchy 了解です
masterにコミットしてから2020でcherry-pickする感じにしましょう
10:02 Kiara Translation 🇬🇧: Confirmed
Commit to master and make it cherry-pick in 2020
10:05 uetchy https://github.com/vivliostyle/vivliostyle.org/pull/17
[vivliostyle/vivliostyle.org] #17 Update viewer version to v2.0.0-pre.8
10:05 Kiara Translation 🇬🇧: https://github.com/vivliostyle/vivliostyle.org/pull/17
🇯🇵: https://github.com/vivliostyle/vivliostyle.org/pull/17
10:07 uetchy /viewer を v2.0.0-pre.8に更新するで間違いないですね?
10:07 Kiara Translation 🇬🇧: It is correct to update / viewer to v2.0.0-pre.8, right?
10:08 shinyu はい、それでよいです。
10:08 Kiara Translation 🇬🇧: Yes, that's fine.
10:09 uetchy マージしました
10:09 Kiara Translation 🇬🇧: Merged
10:12 shinyu ところで、
https://www.npmjs.com/package/@vivliostyle/core
https://www.npmjs.com/package/@vivliostyle/viewer
を見ると、latest が “<https://www.npmjs.com/package/@vivliostyle/core/v/2.0.0-pre.2
10:13 Kiara Translation 🇬🇧: by the way,
https://www.npmjs.com/package/@vivliostyle/core
https://www.npmjs.com/package/@vivliostyle/viewer
Looking at, latest is “<https://www.npmjs.com/package/@vivliostyle/core/v/2.0.0-pre.2
10:14 uetchy ん、テストする時はnpm install @vivliostyle/viewer@next のようにすれば入りますよ(latestタグは正式リリース用)
10:14 Kiara Translation 🇬🇧: Well, when testing, enter npm install @ vivliostyle / viewer @ next (latest tag is for official release)
10:14 uetchy preがlatestに入っているのはなにかのミスですね
10:14 Kiara Translation 🇬🇧: It's a mistake that pre is in latest
10:16 shinyu では気にしないことにします
10:16 Kiara Translation 🇬🇧: So I wo n’t mind
Clone this wiki locally