-
Notifications
You must be signed in to change notification settings - Fork 53
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
Typescript them all #57
base: master
Are you sure you want to change the base?
Conversation
Calamari
commented
Nov 7, 2021
•
edited
Loading
edited
- Add Types to everything
- Experiment sufficiently
- Rewrite README to mention Typescript properly
Hello! How is the TypeScript version going? What are you missing before you can release it? I can use it to maybe give you some feedback if you publish it to npm with any kind of name. (Same for the parallel branch) |
Hello @Telokis. |
Hello and thank you for the answer! If you can, I'll be happy to test the TypeScript version! It's difficult to find a proper behavior tree library in JS and even more so in TypeScript! (I was asking for parallel/async as well because my real AI will use websockets to perform actions but that can come later, I'll do a synchronous prototype in the first time.) |
Tasks should not start the timer as long as they are running.
So I added the parallel branch stuff and this into one release: [email protected] It should be published by now. |
I will also test this new support and let you know what I find. :) |
@Tresky Thanks, very much for trying it out. You are right, during the publishing step, all files except those mentioned have been omitted. 🤦 But it should work now with the next beta release: [email protected] |