Skip to content

[TASK] Use CommentContainer trait to implement Commentable #1217

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

Merged
merged 1 commit into from
Mar 26, 2025

Conversation

JakeQZ
Copy link
Contributor

@JakeQZ JakeQZ commented Mar 25, 2025

Closes #813.

@JakeQZ JakeQZ added cleanup refactor For PRs that refactor code without changing functionality labels Mar 25, 2025
@JakeQZ JakeQZ requested a review from oliverklee March 25, 2025 23:10
@JakeQZ JakeQZ self-assigned this Mar 25, 2025
@@ -4,8 +4,7 @@

namespace Sabberworm\CSS\CSSList;

use Sabberworm\CSS\Comment\Comment;
use Sabberworm\CSS\Comment\Commentable;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know why the fixer didn't complain about this unsed use (CSSListItem extends Commentable, as of #1212, but when that change was made, Commentable was no longer directly referenced in this file).

@oliverklee oliverklee merged commit 8104f52 into main Mar 26, 2025
21 checks passed
@oliverklee oliverklee deleted the task/use-commentcontainer branch March 26, 2025 00:08
@coveralls
Copy link

Coverage Status

coverage: 52.82% (+0.6%) from 52.257%
when pulling e5f29ca on task/use-commentcontainer
into ee07bde on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup refactor For PRs that refactor code without changing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add trait for 'standard' Commentable implementation
3 participants