studio
/
watchtower
3
6
Fork 4
Code Issues 5 Pull Requests Releases Activity

JS->TS: Port project from JavaScript to TypeScript #6

New Issue
Open
opened 2023-07-30 17:54:27 +02:00 by Inês Almeida · 0 comments
Inês Almeida commented 2023-07-30 17:54:27 +02:00
Collaborator
Copy Link

The renderer and watchtower were initially developed in JS, but this led to constant problems and time wasted in debugging mistakes with poor error reporting.
TypeScript should help with typing safer code and having better tools and static code analysis instead of things going 'undefined' sometimes :) Although it's work to convert the full project, it should be a worthy investment considering the amount of developer sadness that went into trivial mistakes.

  • Add TS to the pipeline.
  • Add type hints to the code progressively.
  • Fix errors found by the static analysis.
  • All code in TS.
  • Enable the most pedantic set of errors, no 'any', strict null checks etc.
  • Imports are working correctly.
  • All IDEs and settings enabled. No errors left :)
The renderer and watchtower were initially developed in JS, but this led to constant problems and time wasted in debugging mistakes with poor error reporting. TypeScript should help with typing safer code and having better tools and static code analysis instead of things going 'undefined' sometimes :) Although it's work to convert the full project, it should be a worthy investment considering the amount of developer sadness that went into trivial mistakes. - [x] Add TS to the pipeline. - [x] Add type hints to the code progressively. - [ ] Fix errors found by the static analysis. - [ ] All code in TS. - [ ] Enable the most pedantic set of errors, no 'any', strict null checks etc. - [x] Imports are working correctly. - [ ] All IDEs and settings enabled. No errors left :)
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
v1.0.0
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: studio/watchtower#6
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?