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

Add type hints to move functions #49

Open
Debilski opened this issue Jun 26, 2021 · 2 comments
Open

Add type hints to move functions #49

Debilski opened this issue Jun 26, 2021 · 2 comments

Comments

@Debilski
Copy link
Member

If type hints are not on the schedule for 2021, we won’t add it for now. But having auto completion (+ API docs on hover!) is neat and probably worth considering at some point.

image

@lschwetlick
Copy link
Contributor

are you suggesting doing it only for the demo bots? Or all over the pelita repo as well?

Personally I'm a bit concerned it decreases readability for people who haven't encountered too much typing in their programming experience.

@Debilski
Copy link
Member Author

Debilski commented Jul 2, 2021

Not really all over the Pepita repo (at least not yet?), perhaps for the bot API, but primarily just for the move function, as static analysis figures it out in a lot of cases. In this situation, though, it is of course impossible for the editor to know what bot is supposed to be.
But yeah, if we’re not going to mention it at the school, it might be too confusing.

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

No branches or pull requests

2 participants