-
-
Notifications
You must be signed in to change notification settings - Fork 389
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
feat(core/respec-global): add toHTML() method #3403
Conversation
@@ -40,6 +41,10 @@ class ReSpec { | |||
get ready() { | |||
return this._respecDonePromise; | |||
} | |||
|
|||
async toHTML() { |
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.
Async in case we need it in future
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 can also add it in the future...
Should add a test too 😅 |
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.
Good to have at least a minimal test.
Lets keep this feature for next release. Keep refactors and minor fixes for this release. |
Ok, but don't let them sit for too long (unless this will have consequences downstream). |
Part of https://github.com/w3c/respec/issues/3089