chore: drop dead Hound integration - #58
Merged
Merged
Conversation
Hound is not installed on this repository: the only webhook is Codecov, and PRs #43-#57 show no Hound review, comment, or check. The README badge therefore advertised a review that never runs. .hound.yml dates from the initial commit and is referenced by nothing. SwiftLint runs through the Fastlane lint lane against .swiftlint.yml.
This was referenced Aug 21, 2026
futamura
added a commit
that referenced
this pull request
Aug 21, 2026
UIKitSpec's "prepares blurred snapshots before becoming visible" waited only 0.5 seconds for the blurred image and failed on CI in PRs #47, #58 and #59. A rerun of the same commit passed every time. FluidBlurredBackgroundView produces the image across two async hops: blurQueue runs CIGaussianBlur, then the main queue assigns blurView.image. A loaded runner does not always finish both within 0.5s. This was the only tight timeout in the suite; the other five call sites use the 10 second default and have never flaked. Drop the override so the test uses that default, and assert visibility and alpha again after the wait. The claim the test makes, that the snapshot is prepared while the view is invisible, now rests on those assertions instead of on how long the wait happened to be.
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
目的
README の
Reviewed by Houndbadge が事実に反していたため、Hound 連携を削除する。主要変更点
.hound.ymlを削除根拠
test1 件のみ.hound.ymlは 2019-06-30 の initial commitd5822cd由来 (36 B)。grep -rn -i houndの結果、repo 内から参照ゼロHound の再導入は却下した。Hound の Swift サポートは SwiftLint 依存であり、同じことを CI で直接実行できるため、外部サービス依存を増やす利点がない。
Verification
git diff --checkexit 0image/svg+xml(CI - passing/ codecov /Swift: 6.3 | 6.2/Platforms: iOS)gh api /markdownで単一<p>に 4 個の<a><img>= 横並び 1 行を確認bundle exec fastlane ios lintが.hound.yml不在でfinished successfullySimulator / 実機確認は非対象 (README + config file のみで、transition / layout / gesture / animation / Storyboard / asset / font に非接触)。
補足
調査の副産物として、CI に SwiftLint が無いことが判明した。
.github/workflows/に lint 参照がなく、ci.ymlの jobtestは Fastlane を経由せずxcodebuildを直接実行している。SwiftLint は fastlanelintlane にしか存在しない。本 PR の scope 外のため、別 task として記録済み。