Skip to content

fix: ci setting tune up #17

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 3 commits into from
Apr 24, 2021
Merged

Conversation

tsuyoshicho
Copy link
Contributor

@tsuyoshicho tsuyoshicho commented Apr 24, 2021

fix below

rel #15
close #16

  • remove workflow_dispatch
  • remove vim setup / use env default
  • remove miss GITHUB_TOKEN env
  • add pull_strategy : not need pull (NO-PULL)
    if not NO-PULL set(default), unstage file conflict and fail commit

@k-takata
Copy link
Member

LGTM

@k-takata k-takata merged commit ff3e94b into vim-jp:master Apr 24, 2021
@tsuyoshicho tsuyoshicho deleted the fix/ci-slimup-20210424 branch April 24, 2021 03:49
@k-takata
Copy link
Member

  がごっそり削られて大量の差分が出ています。 d5c97f0
Vim のバージョンを変えたせいだろうか?

@tsuyoshicho
Copy link
Contributor Author

な、なんでしょう。
処理本体は変えてないので、あるとしたらたしかにそこらなのでしょうけど....

@k-takata
Copy link
Member

ubuntu-latest (20.04) に入っている 8.1.2269 と 8.2.0020 で関係しそうな差分は 8.1.2312 の直前の runtime file update で入った TOhtml の更新ですかね?
TOhtml のその前の更新は 8.1.0528の直前。
ubuntu-latest に入っている TOhtml がバグってる可能性があるかな?(ざっと見た感じ、それっぽい箇所は見当たらなかったのだが)

@k-takata
Copy link
Member

Ubuntu 20.04 の vim に最新の TOhtml を組み合わせたら、以前と同じ結果が得られました。
なので、ubuntu-latest に入っている TOhtml がバグってるものと思われます。
Vim は 8.2 に戻した方が良さそうです。

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.

ci: GHA can be slim up?
2 participants