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

Fix broken CI badge: Travis CI to GitHub Actions #123

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

r7kamura
Copy link
Contributor

@r7kamura r7kamura commented Jan 2, 2024

概要

TravisCIのバッジが参照されているのを見つけたので、GitHub Actionsのものを使うように変更する。

詳細

変更前

変更前は、下図のような状態だった。

image

変更後

変更後は、下図のような状態になる。

image

歴史

元々、以下のcommitでCIがTravis CIからGitHub Actionsに変更された。

その他

https://github.com/textlint-ja 配下のリポジトリを見ると、バッジ画像のalt属性には概ね Actions Status: test という文字列が設定されている場合が多いため、今回はこれを踏襲することにした。異なるケースとして、https://github.com/textlint-ja/textlint-rule-preset-ja-technical-writing では Actions Status が設定されている。

バッジに割り当てられるa要素のhref属性について、既存のリポジトリを見ると、workflow:test というクエリでの検索結果を表すURLが割り当てられている。一方、GitHub ActionsのバッジURL生成機能では、個々のworkflowを一意に表すURLが用いられる。この2つは表示内容に異なる部分があり、例えば後者のURLではサイドバーのUIで現在test workflowのページを閲覧していることが分かる。今回このPull Requestでは、後者を採用することにした。

Copy link

changeset-bot bot commented Jan 2, 2024

⚠️ No Changeset found

Latest commit: 08ef1bf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@azu azu merged commit 600d1e4 into textlint-ja:master Jan 2, 2024
2 checks passed
@r7kamura r7kamura deleted the readme-ci-badge branch January 2, 2024 08:14
@azu
Copy link
Member

azu commented Jan 2, 2024

Thanks

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.

2 participants