Skip to content
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

CI: Check that Vim help tags in runtime folder are correct #1426

Merged
merged 1 commit into from
Sep 9, 2023

Conversation

ychin
Copy link
Member

@ychin ychin commented Sep 8, 2023

Previously, Vim didn't always have up-to-date help tags. As a result, we just manually generate the help tags when publishing a release (#1286), even though it doesn't actually make sure the tags file in source is actually accurate. However, Vim has since added CI checks to make sure the help tags are correct, so we can change MacVim CI to do the same thing as well. This way, the source version is always correct and we won't rely on a build step during release to fix it, and it also makes sure other methods to get MacVim (build from source, Homebrew) will be correct as well.

@github-actions github-actions bot added the CI Vim upstream label for CI issues label Sep 8, 2023
@ychin ychin added Infrastructure Non-app infrastructure issues, e.g. CI Non User Facing Non-user facing change. These issues do no need to show up in release notes. CI Vim upstream label for CI issues and removed CI Vim upstream label for CI issues labels Sep 8, 2023
@ychin ychin force-pushed the check-vimtags-in-ci branch 2 times, most recently from 41030d6 to 75f9f1e Compare September 9, 2023 09:52
Previously, Vim didn't always have up-to-date help tags. As a result, we
just manually generate the help tags when publishing a release (macvim-dev#1286),
even though it doesn't actually make sure the tags file in source is
actually accurate. However, Vim has since added CI checks to make sure
the help tags are correct, so we can change MacVim CI to do the same
thing as well. This way, the source version is always correct and we
won't rely on a build step during release to fix it, and it also makes
sure other methods to get MacVim (build from source, Homebrew) will be
correct as well.
@ychin ychin merged commit 7b8a803 into macvim-dev:master Sep 9, 2023
4 checks passed
@ychin ychin deleted the check-vimtags-in-ci branch September 9, 2023 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Vim upstream label for CI issues Infrastructure Non-app infrastructure issues, e.g. CI Non User Facing Non-user facing change. These issues do no need to show up in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant