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

chore!: remove @ts-ignore syntax [skip release] #57

Merged
merged 2 commits into from
May 7, 2024

Conversation

dsanders11
Copy link
Member

Chained on top of #56.

@ts-expect-error is a much better solution, so remove @ts-ignore as it's not providing any value.

@dsanders11 dsanders11 requested a review from a team as a code owner May 3, 2024 00:41
@dsanders11 dsanders11 marked this pull request as draft May 3, 2024 00:41
@dsanders11 dsanders11 force-pushed the feat/generic-ts-check branch from 1af6a1a to 4ab5491 Compare May 3, 2024 00:47
@dsanders11 dsanders11 force-pushed the chore/remove-ts-ignore branch 2 times, most recently from df0a2ab to 2098323 Compare May 4, 2024 04:22
@dsanders11 dsanders11 force-pushed the feat/generic-ts-check branch from 4ab5491 to acaf0b2 Compare May 4, 2024 04:30
@dsanders11 dsanders11 force-pushed the chore/remove-ts-ignore branch from 2098323 to 7d2f5e3 Compare May 4, 2024 04:33
Base automatically changed from feat/generic-ts-check to main May 7, 2024 17:49
BREAKING CHANGE: removed @ts-ignore syntax for lint-markdown-ts-check
@dsanders11 dsanders11 force-pushed the chore/remove-ts-ignore branch from 7d2f5e3 to 14f2ca8 Compare May 7, 2024 17:55
@dsanders11 dsanders11 marked this pull request as ready for review May 7, 2024 17:55
@dsanders11 dsanders11 force-pushed the chore/remove-ts-ignore branch from bedf5c9 to 68e40fe Compare May 7, 2024 18:11
@dsanders11 dsanders11 merged commit 212d3cd into main May 7, 2024
13 checks passed
@dsanders11 dsanders11 deleted the chore/remove-ts-ignore branch May 7, 2024 18:19
Copy link

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants