-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
docs: add changelog note about typescript #1414
Conversation
CHANGELOG.md
Outdated
@@ -7,6 +7,7 @@ Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CO | |||
* Add support for Gherkin's [Rule/Example syntax](https://cucumber.io/docs/gherkin/reference/#rule) | |||
* Add `transpose` method to [data table interface](docs/support_files/data_table_interface.md) | |||
* Add `log` function to world, providing a shorthand to log plain text as [attachment(s)](docs/support_files/attachments.md) | |||
* `cucumber-js` is now written in [TypeScript](https://www.typescriptlang.org/) and includes its own typings, so you no longer need to depend on `@types/cucumber` in your own TypeScript projects |
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.
I think this could be shortened to:
Now includes
Typescript
type definitions, deprecating the need for@types/cucumber
in TypeScript projects
Thoughts?
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.
Agree that's better, updated.
I had a look at the tests from DefinitelyTyped and ran them with tsd, there are quite a few inaccuracies (e.g. things like Around
and defineSupportCode
that we don't export, plus some naming differences of classes and interfaces. Not sure if you think we need to call it out in the changelog, but I think TypeScript users have probably come to expect this when a library starts publishing its own types.
@charlierudolph all okay on this one now? |
* docs: add changelog note about typescript * shorten message
Not sure if we should call out any differences between
@types/cucumber
and our own typings. For exampleTableDefinition
->DataTable
.I started looking at using tsd to test the example code from DefinitelyTyped to expose the differences, going to continue with that today.
Fixes #1269