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
elem: Fix lint and generate issues #205
Conversation
Uses a couple regular expressions, and a couple special cases for <audio> and <h{1-6}> now.
Codecov Report
@@ Coverage Diff @@
## master #205 +/- ##
=======================================
Coverage 52.22% 52.22%
=======================================
Files 3 3
Lines 653 653
=======================================
Hits 341 341
Misses 258 258
Partials 54 54 Continue to review full report at Codecov.
|
The first one is replaced with funName, the rest were not added back and were blank. This is a bug fix.
elem/elem.gen.go
Outdated
// | ||
// https://developer.mozilla.org/en-US/docs/Web/HTML/Element/sub | ||
func Subscript(markup ...vecty.MarkupOrChild) *vecty.HTML { | ||
return vecty.Tag("sub", markup...) | ||
} | ||
|
||
// Summary is used as a summary, caption, or legend for the content of a | ||
// <details> element. | ||
// Summary element specifies a summary, caption, or legend for a <details> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll need to fix this to remove the word element
or make it a proper sentence somehow.
elem/elem.gen.go
Outdated
// Template is a mechanism for holding client-side content that is not to be | ||
// rendered when a page is loaded but may subsequently be instantiated during | ||
// runtime using JavaScript. | ||
// Template element is a mechanism for holding client-side content that is not |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
aside from inline comments, I am fine with merging this. |
Good catch, there were several of the same type of pattern that weren't matched/corrected. The diff on the last commit seems to only get rid of the extra "element" you commented about. Yay. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
CI currently fails on the
elem
package due to some function comments not being manipulated as needed to satisfygo vet
. Also, runninggo generate
on the elem package generated invalid code because of changes to the MDN website. This PR fixes those issues and touches up some MDN description irregularities.