Skip to content

chore(CI): Add upstream sync script #151

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 2 commits into from
Nov 3, 2020
Merged

Conversation

potato4d
Copy link
Member

@potato4d potato4d commented Nov 1, 2020

Description of Problem

本家の内容を一度に取り込んでしまいたいが、 #150 の方針を取るために定期的に本家の内容を master に反映しておき、 lang-ja -> master への rebase のサイズ感を確認したい。

手動でやっても良いが、忘れかねない。

Proposed Solution

定期的に master の内容を更新する GitHub Actions を用意した。

Additional Information

GitHub Actions の cron は only running at default branch なので、コードにミスはあるかもしれない

@potato4d
Copy link
Member Author

potato4d commented Nov 1, 2020

@kazupon @egurinko
こちらお願いしますー

@netlify
Copy link

netlify bot commented Nov 1, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit bcebc36

https://deploy-preview-151--vuejs-v3-ja-doc-preview.netlify.app

Copy link
Member

@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

LGTM!

node-version: [12.x]

steps:
- uses: actions/checkout@v1
Copy link
Member

Choose a reason for hiding this comment

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

あ、
actions/checkout 、v2 が使えると思うんですけど、しない理由ってなんでしたっけ?

Copy link
Member Author

Choose a reason for hiding this comment

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

あー、単純に手元にあった同じ要件を解決するための v1 時代のコードを書き換えただけだからです 🙇
v2にしておきます 🙏

@potato4d potato4d merged commit 39e133f into lang-ja Nov 3, 2020
@potato4d potato4d deleted the chore/sync-upstream branch November 3, 2020 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants