Skip to content

fix(types): mark datetime on <del> as optional#4570

Merged
marvinhagemeister merged 1 commit intomainfrom
del-datetime-optional
Nov 25, 2024
Merged

fix(types): mark datetime on <del> as optional#4570
marvinhagemeister merged 1 commit intomainfrom
del-datetime-optional

Conversation

@marvinhagemeister
Copy link
Copy Markdown
Member

In #4554 the datetime attribute of <del> was accidentally marked as required. This PR fixes that.

Copy link
Copy Markdown
Member

@rschristian rschristian left a comment

Choose a reason for hiding this comment

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

Crap, sorry!

@marvinhagemeister
Copy link
Copy Markdown
Member Author

No worries, could've happened to anyone

@marvinhagemeister marvinhagemeister merged commit bd14437 into main Nov 25, 2024
@marvinhagemeister marvinhagemeister deleted the del-datetime-optional branch November 25, 2024 10:54
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 99.488%. remained the same
when pulling c7ae2d2 on del-datetime-optional
into 269cdd8 on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants