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

Don't inline classes in constructor if possible #481

Merged
merged 14 commits into from
Nov 4, 2024

use non-syntatic class names in tests

44ce813
Select commit
Loading
Failed to load commit list.
Merged

Don't inline classes in constructor if possible #481

use non-syntatic class names in tests
44ce813
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 4, 2024 in 0s

94.33% of diff hit (target 86.29%)

View this Pull Request on Codecov

94.33% of diff hit (target 86.29%)

Annotations

Check warning on line 47 in R/base.R

See this annotation in the file changed.

@codecov codecov / codecov/patch

R/base.R#L47

Added line #L47 was not covered by tests

Check warning on line 137 in R/class-spec.R

See this annotation in the file changed.

@codecov codecov / codecov/patch

R/class-spec.R#L137

Added line #L137 was not covered by tests

Check warning on line 158 in R/class-spec.R

See this annotation in the file changed.

@codecov codecov / codecov/patch

R/class-spec.R#L158

Added line #L158 was not covered by tests