Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support version up to Vim 9.1 and drop Vim 8.2. #828

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

tsuyoshicho
Copy link
Contributor

@tsuyoshicho tsuyoshicho commented Dec 31, 2024

Vim 9.1 / 9.0 Support へ変更するPRです。

サポートバージョンの変更対応です。

対象バージョンが変化するため、CIでのrequiredで8.2はマージ後取り除き、9.1系を設定する必要があります。
(このPRでは8.2系は常にひっかかります)

変更として、必要最小の対応です。

いくつかの機能は高度化の可能性がありますが、それは別途個別に行います。

@tsuyoshicho tsuyoshicho self-assigned this Dec 31, 2024
@tsuyoshicho
Copy link
Contributor Author

テストや実装の修正もありますので、そろったらまたレビューをおねがいします @ujihisa

@@ -10,25 +10,21 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
vim: [v8.2.0020, v9.0.0107, head]
vim: [v9.0.0107, v9.1.0100 ,head]
Copy link
Contributor Author

@tsuyoshicho tsuyoshicho Jan 2, 2025

Choose a reason for hiding this comment

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

9.1.0000 は linux で色データのマッチが不正になってしまうので、9.1.0100 としました

  • 現在 runtime/colors/lists/default.vim に色データはあります(lowercase)
  • Color.vim は検索時にlowercaseに整理してして行います
  • データとしてYellow(yellow)は常にありますが、失敗しています

問題が修正されたパッチを見付けれられなかったので、ほどほど後のバージョンをとりあえずで決めて入れています

see link

@tsuyoshicho tsuyoshicho removed the WIP label Jan 2, 2025
@tsuyoshicho tsuyoshicho marked this pull request as ready for review January 2, 2025 11:05
Copy link
Member

@ujihisa ujihisa left a comment

Choose a reason for hiding this comment

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

偉業!ありがとございます

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

Successfully merging this pull request may close these issues.

2 participants