Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

ライセンスのMarkdownからHTMLを書き出すスクリプトを追加 #904

Merged
2 commits merged into from
Mar 9, 2022

Conversation

ghost
Copy link

@ghost ghost commented Mar 8, 2022

Issue 番号 / Issue ID

  • Close #

目的 / Purpose

  • ライセンスのMarkdownからHTML書き出しを簡単に行えるようにしたい

変更内容 / Changes

  • Python-Markdownを使用してHTML部分を書き出す
    • もともとSublime TextのMarkdownPreviewプラグインを使用して書き出したようなので、このプラグインで使用していると思われるPython-Markdownを使用した
    • Sublime TextはFree版をずっと使い続けられないようなので、これに依存せずに書き出せるようにした
  • 当スクリプトを使用する前に予めPython-Markdownをインストールし使用できるようにしておく
  • 書き出し結果はTimeZoneConverter部分で差異がある
    • Markdownでは記載されていて表示されていなかった内容が表示されるようになった
  • macOS Big Sur で確認済み

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

確認事項 / What to check

  • [ ]

その他 / Other information

  • 使い方
    $ ./Tools/app-generate-license-html.sh

Internal IDs:

  • Task 6389

@ghost ghost requested review from a user, cocoa-dev003 and cocoa-dev005 March 8, 2022 07:30
@ghost ghost self-assigned this Mar 8, 2022
@ghost ghost requested a review from keiji March 8, 2022 07:36
Copy link
Collaborator

@keiji keiji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

手動でHTMLを書き出さなくてもいいのはいいですね!

CIで常に最新のファイルをビルドして置き換えるようにするもよし、手元で手動で生成して配置するもよし、どちらにせよコマンドラインでHTML生成できるようにするのは良いアイデアです。

Tools/app-generate-license-html.sh Show resolved Hide resolved
@ghost ghost force-pushed the feature/app-generate-license-html branch from 173c75e to 88d0e0b Compare March 8, 2022 07:56
@ghost ghost merged commit 1a4329e into develop Mar 9, 2022
@ghost ghost deleted the feature/app-generate-license-html branch March 9, 2022 02:37
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants