Skip to content
This repository has been archived by the owner on Sep 21, 2022. It is now read-only.

Migration to TypeScript #1

Open
wants to merge 4 commits into
base: trunk
Choose a base branch
from

Conversation

tedraykov
Copy link

@tedraykov tedraykov commented Oct 2, 2021

Impact: minor
Type: refactor

Issue

The current API is using vanilla JavaScript which lacks type checking.

Solution

A migration to TypeScript will provide a better development experience.

Testing

  1. Run reaction API with the modified plugin

Co-authored-by: ts-migrate <>
Signed-off-by: tedraykov <tedraykov@gmail.com>
Co-authored-by: ts-migrate <>
Signed-off-by: tedraykov <tedraykov@gmail.com>
Co-authored-by: ts-migrate <>
Signed-off-by: tedraykov <tedraykov@gmail.com>
Signed-off-by: tedraykov <tedraykov@gmail.com>
@Akarshit
Copy link

Akarshit commented Oct 2, 2021

Wow! This is awesome!

@tedraykov
Copy link
Author

tedraykov commented Oct 2, 2021

In order to make new API plugins or migrate the existing ones to TS, this repo, reaction logger and api-utils have to be migrated. After that come the api-core.

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

Successfully merging this pull request may close these issues.

2 participants