WIP: Rustification: Add NAPI-RS transition-rust-backend with capnp se…#1434
WIP: Rustification: Add NAPI-RS transition-rust-backend with capnp se…#1434greenscientist wants to merge 1 commit intochairemobilite:mainfrom
Conversation
tahini
left a comment
tahini
left a comment
There was a problem hiding this comment.
Je sais que c'est en draft, mais j'ai regardé vite fait c'était quoi cette PR. Finalement, c'est un premier exemple de comment faire faire des choses à Rust par nodejs? Donc éventuellement des calculs de tout genre... Et si on a un point d'entrée pour le faire facilement, ce sera plus simple de décider de passer de nouvelles fonctionnalités à Rust.
| "packages/transition-backend", | ||
| "packages/transition-frontend" | ||
| "packages/transition-frontend", | ||
| "packages/transition-rust-backend" |
There was a problem hiding this comment.
transition-rust-backend demeure un package nodejs? Aussi, ces packages sont exécutés dans l'ordre, si je comprends bien les dépendences, il faudrait qu'il soit avant chaire-lib-backend ici.
There was a problem hiding this comment.
tu veux dire avant transition-backend ?
| @@ -0,0 +1,197 @@ | |||
| # Created by https://www.toptal.com/developers/gitignore/api/node | |||
| # Edit at https://www.toptal.com/developers/gitignore?templates=node | |||
There was a problem hiding this comment.
a-t-on beosin d'un fichier .gitignore spécifique pour ce package?
There was a problem hiding this comment.
Plus clean de garder les trucs rust ici à mon avis
| ) | ||
| try { | ||
| if (localFileExisted) { | ||
| nativeBinding = require('./transition-rust-backend.linux-arm64-gnu.node') |
There was a problem hiding this comment.
Où sont/seront tous ces fichiers? Ce sera compilé? ./ ça veut dire que tout se retrouvera dans le root directory du package? Il faudrait idéalement envoyer tous ces fichiers dans un même répertoire de compilation.
There was a problem hiding this comment.
Tout ca est auto généré par napi-rs, donc je sais pas à quel point on peut modifier le tout. a voir.
2 participants
…rialization