-
Notifications
You must be signed in to change notification settings - Fork 155
Fixes to TypeScript type definition file #297
Conversation
🐛 fix: TypeScript type definition file was not included in package.json. fixes #296
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 so much for this!
test.ts
Outdated
|
||
console.assert(typeof Terraformer !== undefined); | ||
|
||
let point = new Terraformer.Primitive({ | ||
type: "Point", | ||
coordinates: [1, 2] | ||
coordinates: [1, 2], |
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.
are the new trailing commas added automatically?
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.
once we clean them up, i think we'll be good to merge.
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've added a TSLint test to npm run test:ts
. The tslint config has been modified to disallow the trailing commas. (By default it requires them. I have no preference either way on this matter.)
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.
you're a hero. i'll find time to pull this down and kick the tires in the next couple of days.
Modified TypeScript files to pass TSLint test. Modified TSLint configuration.
Fixes #296