Skip to content

ci: Github Action workflow define : push vimdoc-en gh-pages #15

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 4 commits into from
Apr 22, 2021

Conversation

tsuyoshicho
Copy link
Contributor

ref vim-jp/vimdoc-ja-working#901

定期実行で vimdoc-en の gh-pages を更新します。

成果物

- master
workflow_dispatch:
schedule:
- cron: '36 20 * * Sat'
Copy link
Member

Choose a reason for hiding this comment

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

以前は1日1回実行していたのですが、週1ですか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

あー、すみません。元のTravisの設定から読めてなかったので、仮で入れてた面あります。(出すまでに考慮のつもりが忘れさられてた)
dailyに修正します。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

発動タイミングを、既存のログから同じ時間帯でdailyにしました。

Copy link
Member

Choose a reason for hiding this comment

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

Travisのcronは実際にはcronではなかったので、発動タイミングが徐々に遅くなるという代物でした。
なので、実際のところ、元の発動タイミングに意図はありませんw

@k-takata k-takata merged commit ba73ee0 into vim-jp:master Apr 22, 2021
@k-takata
Copy link
Member

thx

@tsuyoshicho tsuyoshicho deleted the feature/gha-generate-20210416 branch April 22, 2021 22:54
Copy link
Member

@k-takata k-takata left a comment

Choose a reason for hiding this comment

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

見直していたらちょっと気になるところがありました。

push:
branches:
- master
workflow_dispatch:
Copy link
Member

Choose a reason for hiding this comment

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

workflow_dispatch は必要ないですよね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

手動生成できるので、動作確認にいいかなと思いましたが、削除しますか?

Copy link
Member

Choose a reason for hiding this comment

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

どういう使い方を想定されていますか?
最新のを実行し直すだけなら、最新のscheduled buildをre-runしても良さそうですが。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

そうですね...
re-runとしてではなく、新規にrunしたほうがいい場合があるかなと思ったというのはありますが、差がないですね。

Copy link
Member

Choose a reason for hiding this comment

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

まー、re-runだと前のログが消えてしまうという問題はありますが…

Copy link
Contributor Author

Choose a reason for hiding this comment

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

あー、いまさら & このリポジトリだと意味がないのですが、思いだしました。

GitHub workflowのcronは、リポジトリ更新がないと60日で止るのですが
再開時に即座に更新を手動でしたいならあったほうがいいかな、的に考えてました。

まあ、不要なのは変りませんね(Vimの状況からすると)

Comment on lines +48 to +49
env:
GITHUB_TOKEN: ${{ secrets.VIMDOC_JP_TOKEN }}
Copy link
Member

Choose a reason for hiding this comment

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

この行、もしかして不要ですか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ですね、すみません。

@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.

2 participants