Skip to content

Conversation

@JakeQZ
Copy link
Collaborator

@JakeQZ JakeQZ commented Jan 4, 2026

Note that ValueList is an abstract class. The implementation can be reused by the concrete subclasses. In the case of LineName and RuleValueList it can be inherited as is, because they don't have any additional properties. CSSFunction will need to add the name property.

For now, exception-throwing stubs are added to the subclasses, pending implementation on a case-by-case basis.

Part of #1440.

Note that `ValueList` is an abstract class.  The implementation can be reused
by the concrete subclasses.  In the case of `LineName` and `RuleValueList` it
can be inherited as is, because they don't have any additional properties.
`CSSFunction` will need to add the `name` property.

For now, exception-throwing stubs are added to the subclasses, pending
implementation on a case-by-case basis.

Part of #1440.
@JakeQZ JakeQZ requested a review from oliverklee January 4, 2026 00:30
@JakeQZ JakeQZ self-assigned this Jan 4, 2026
@JakeQZ JakeQZ added testing PRs/issues adding additional tests only, or primarily testing-focused developer-specific Issues that only affect maintainers, contributors, and people submitting PRs labels Jan 4, 2026
@coveralls
Copy link

Coverage Status

coverage: 70.305% (+0.2%) from 70.077%
when pulling b3aba8f on task/get-array-rep/valuelist
into b6e4d25 on main.

@oliverklee oliverklee merged commit 0a2c875 into main Jan 4, 2026
23 checks passed
@oliverklee oliverklee deleted the task/get-array-rep/valuelist branch January 4, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer-specific Issues that only affect maintainers, contributors, and people submitting PRs testing PRs/issues adding additional tests only, or primarily testing-focused

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants