Skip to content

Commit

Permalink
use Vim 9.1.0626 for checking installing specific version of Vim
Browse files Browse the repository at this point in the history
  • Loading branch information
rhysd committed Jul 28, 2024
1 parent 00d8987 commit a2e3c69
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ jobs:
run: node ./scripts/post_action_check.js "${{ matrix.neovim }}" "${{ matrix.version }}" "${{ steps.vim.outputs.executable }}"

# separate from stable-and-nightly since jobs.{id}.name.strategy.matrix.exclude seems not working
vim-v8_2_1931:
name: Vim v8.2.1931
vim-v9_1_0000:
name: Vim v9.1.0626
strategy:
matrix:
os: [ubuntu-latest, macos-latest, windows-latest]
Expand All @@ -44,10 +44,10 @@ jobs:
- run: npm run build
- uses: ./
with:
version: v8.2.1931
version: v9.1.0626
id: vim
- name: Validate action result
run: node ./scripts/post_action_check.js "false" "v8.2.1931" "${{ steps.vim.outputs.executable }}"
run: node ./scripts/post_action_check.js "false" "v9.1.0626" "${{ steps.vim.outputs.executable }}"

nvim-v0_4_4:
name: Neovim v0.4.4
Expand Down
6 changes: 3 additions & 3 deletions scripts/post_action_check.ts
Original file line number Diff line number Diff line change
Expand Up @@ -120,15 +120,15 @@ function main(): void {
const m = args.version.match(/^v(\d+\.\d+)\.(\d+)$/);
ok(m);
const major = m[1];
const patch = m[2];
const patch = parseInt(m[2], 10);

const l = `VIM - Vi IMproved ${major}`;
ok(stdout.includes(l), `First line '${l}' should be included in stdout: ${stdout}`);

// assert.match is not available since it is experimental
ok(
stdout.match(new RegExp(`Included patches: .*${patch}`)),
`Patch ${patch} should be included in stdout: ${stdout}`,
stdout.includes(`Included patches: 1-${patch}`),
`Patch 1-${patch} should be included in stdout: ${stdout}`,
);
}
} else {
Expand Down

0 comments on commit a2e3c69

Please sign in to comment.