padarom commented on PR #896:
URL: https://github.com/apache/guacamole-client/pull/896#issuecomment-1796177638

   The intention of my fork was to make it easier for people to use Guacamole 
in their JS projects, so if there ends up being an official Apache package 
doing the same, then my fork can be seen as obsolete.
   
   As long as the official npm package is compatible, I'm fine with 
transferring ownership. If I understood correctly the plan seems to include a 
major version bump, so even if it isn't quite compatible some breakage could be 
expected by the package's current user base anyways.
   
   GitHub's diff viewer isn't being nice to me with this large PR and I'm 
currently on the go, but from the PR description it seems like the regular 
Guacamole frontend is rewritten in Typescript, whereas `guacamole-common-js` is 
not. Are type definitions still expected to be provided via 
`@types/guacamole-common-js`? If I could make the recommendation to also think 
about including (and bundling) types directly in the official package, rather 
than having users manually add another package that has to be separately 
maintained?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@guacamole.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to