Skip to content

Update lint rules, force testify/assert#192

Merged
RyanPotat merged 1 commit intomasterfrom
RyanPotat/lint/test-helpers
Mar 29, 2025
Merged

Update lint rules, force testify/assert#192
RyanPotat merged 1 commit intomasterfrom
RyanPotat/lint/test-helpers

Conversation

@RyanPotat
Copy link
Copy Markdown
Member

Description

Update lint rules to forbid the use of direct testing.T helpers.

Reference issue

pion/.goassets/pull/222

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.70%. Comparing base (7752cd4) to head (bb27ec6).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   77.70%   77.70%           
=======================================
  Files          22       22           
  Lines        2548     2548           
=======================================
  Hits         1980     1980           
  Misses        472      472           
  Partials       96       96           
Flag Coverage Δ
go 77.70% <ø> (ø)
wasm 77.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RyanPotat RyanPotat force-pushed the RyanPotat/lint/test-helpers branch from 9264d6d to 4e77826 Compare March 28, 2025 22:54
@JoTurk
Copy link
Copy Markdown
Member

JoTurk commented Mar 29, 2025

THANK YOU RYAN <3

@RyanPotat RyanPotat merged commit bb27ec6 into master Mar 29, 2025
15 checks passed
@RyanPotat RyanPotat deleted the RyanPotat/lint/test-helpers branch March 29, 2025 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants