Skip to content
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

fix(types): Add noHtml option #2810

Merged
merged 2 commits into from
Feb 18, 2021
Merged

fix(types): Add noHtml option #2810

merged 2 commits into from
Feb 18, 2021

Conversation

dylanb
Copy link
Contributor

@dylanb dylanb commented Feb 17, 2021

was missed in the 4.1.2 release

Closes #2793

@dylanb dylanb requested a review from a team as a code owner February 17, 2021 13:03
Copy link
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

Seems the indentation is off. I would have expected prettier to catch that but seems it did not.

@stephenmathieson stephenmathieson changed the title Add noHtml to the typescript types fix(types): Add noHtml option Feb 17, 2021
@WilcoFiers WilcoFiers merged commit c03c826 into develop Feb 18, 2021
@WilcoFiers WilcoFiers deleted the nohtml-typescript branch February 18, 2021 10:53
straker pushed a commit that referenced this pull request Mar 2, 2021
* Add noHtml to the typescript types

* Update axe.d.ts

Co-authored-by: Stephen Mathieson <me@stephenmathieson.com>
straker added a commit that referenced this pull request Mar 4, 2021
* fix(types): Add noHtml option (#2810)

* Add noHtml to the typescript types

* Update axe.d.ts

Co-authored-by: Stephen Mathieson <me@stephenmathieson.com>

* fix(respondable): Avoid message duplication with messageId (#2816)

* refactor: rewrite respondable

* fix: use nodejs crypto for uuid.v4

* test(cmd): allow require('crypto')

* tests: fix respondable tests

* remove failing tests

* chore(release): 4.1.3

Co-authored-by: Dylan Barrell <dylan@barrell.com>
Co-authored-by: Stephen Mathieson <me@stephenmathieson.com>
Co-authored-by: Wilco Fiers <WilcoFiers@users.noreply.github.com>
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.

Add noHtml to axe.d.ts
4 participants