Skip to content

ci: Github Action workflow define : push vimdoc-ja master & gh-pages #901

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

Merged
merged 8 commits into from
Apr 25, 2021

Conversation

tsuyoshicho
Copy link
Contributor

@tsuyoshicho tsuyoshicho commented Apr 15, 2021

GitHub workflowを利用して vimdoc-ja-working から vimdoc-ja の更新を行う PR です。

認証にrepoのR/Wのアクセス権のあるトークンを作成し、 vimdoc-ja-working リポジトリの設定 secrets に VIMDOC_JA_TOKEN として設定する必要があります。
(SSH鍵をそのままいくにしても、GitHubのsecretsへの登録など、追加作業が必要になるためToken式に一旦してあります)

成果物

@tsuyoshicho tsuyoshicho changed the title ci: Github Action workflow define : push vimdoc-jp master & gh-pages ci: Github Action workflow define : push vimdoc-ja master & gh-pages Apr 16, 2021
Comment on lines +24 to +28
- name: Setup Vim
uses: thinca/action-setup-vim@v1
with:
vim_version: 'v8.2.0020'
vim_type: 'Vim'
Copy link
Member

Choose a reason for hiding this comment

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

これらの行は syntax-vim-ex にはありませんが、make内でvimを起動できています。
vimdoc-ja-working や vimdoc-en では必要な理由はなんでしょうか?
(できるだけ他(特に個人)への依存は減らしたいです)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

返事は4/23 19:00 +09:00ごろ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

あー、syntax-vim-exでのVimを見過していました
たしかにそうですね

https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-README.md

記載がないからバージョン不明ですが、問題ないなら外しますね
確認するので少々おまちください。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

確認し、削除しました。

またあわせて微修正(記述など)を更新、PRのDescriptionも更新しています。

Copy link
Contributor Author

@tsuyoshicho tsuyoshicho Apr 25, 2021

Choose a reason for hiding this comment

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

ref vim-jp/vimdoc-en#17 (comment)

他の変更は維持しつつ、Vimのバージョンについては8.2系を使うように戻しました。

syntax-vim-exはTOHtmlは使わないので、そのままで大丈夫だと思います。(とりあえずは)

@tsuyoshicho
Copy link
Contributor Author

マージの成否の確認もあるので、マージする前にsecretにTOKENを設定するのがいいと思います
権限は自分はrepoの全部にしていますが、調整できるならそこはおまかせで

@k-takata
Copy link
Member

token を設定したので、試してみましょう。

@k-takata k-takata merged commit f65e9d2 into vim-jp:master Apr 25, 2021
@tsuyoshicho tsuyoshicho deleted the feature/gha-generate-20210414 branch April 25, 2021 01:51
@tsuyoshicho
Copy link
Contributor Author

まずは成功しているようです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants