Skip to content
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

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Typescript them all #57

wants to merge 24 commits into from

Conversation

Calamari
Copy link
Owner

@Calamari Calamari commented Nov 7, 2021

  • Add Types to everything
  • Experiment sufficiently
  • Rewrite README to mention Typescript properly

@Calamari Calamari mentioned this pull request Nov 7, 2021
@Telokis
Copy link

Telokis commented Dec 3, 2022

Hello!
I'm very interested in using this library but I'm unsure whether it's still maintained.

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)

@Calamari
Copy link
Owner Author

Calamari commented Dec 5, 2022

Hello @Telokis.
It is still maintained. I just didn't came around to fully test the TypeScript version to my hearts content. And I don't want to release a version that is broken for some people. But I probably can but it out later as a release candidate. 👍
If you like to play with it, I am happy to do that. You are the first one to ask for a TypeScript version :)

@Telokis
Copy link

Telokis commented Dec 5, 2022

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!
Just let me know the tag once you've published it to npm and I'll start using the thing to design my AI. It's the first time I use a BT so it probably won't be insane but it's still something!

(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.)

@Calamari
Copy link
Owner Author

Calamari commented Dec 5, 2022

So I added the parallel branch stuff and this into one release: [email protected]

It should be published by now.

@Tresky
Copy link

Tresky commented Jan 20, 2023

I will also test this new support and let you know what I find. :)

@Tresky
Copy link

Tresky commented Jan 20, 2023

Just installed the beta version you released. It seems the index.d.ts file did not make it into the build.
Screen Shot 2023-01-20 at 2 29 03 PM

@Calamari
Copy link
Owner Author

@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]
Sorry, for that.

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

Successfully merging this pull request may close these issues.

3 participants