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 annotation for response #28

Closed
odanado opened this issue Mar 18, 2021 · 5 comments · Fixed by #77
Closed

Add type annotation for response #28

odanado opened this issue Mar 18, 2021 · 5 comments · Fixed by #77

Comments

@odanado
Copy link
Contributor

odanado commented Mar 18, 2021

There are no type annotations in backlog.ts. This detracts from the developer experience.

I am interested in this task. Can I work on it?

@mmktomato
Copy link
Member

Sure!
Do you mean about return types? Parameters have types already. https://github.com/nulab/backlog-js/blob/master/src/backlog.ts

@odanado
Copy link
Contributor Author

odanado commented Mar 21, 2021

Do you mean about return types?

Yes.
But, most of the return types are of type any, which can be confusing for developers.

@mmktomato
Copy link
Member

Good.
Let me know if you have any problems on the implementation.

@mmktomato
Copy link
Member

I'm working on this issue. This will come with some breaking changes.

mmktomato pushed a commit that referenced this issue May 23, 2023
mmktomato added a commit that referenced this issue May 31, 2023
@mmktomato
Copy link
Member

Done.
The details of breaking changes: #77 (comment)

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 a pull request may close this issue.

2 participants