Tierney Cyren
Tierney Cyren
Yay! If you have any questions @gireeshpunathil, more than happy to do what I can to help get you going 👍
that sounds good to me, though I'm not sure if fetching data from a database is meaningfully testable in CI. More than happy to be wrong!
sounds good to me :)
@iandrc The intent of this repository is to include many different kinds of examples. I'd look at it less as a pros/cons comparison and more of a way to demonstrate...
Will take a more in-depth look soon here (probably Monday) but one tiny piece of feedback is that you're missing `README.md` for the added directories (see the [countEntriesInDirectory](https://github.com/nodejs/examples/tree/master/cli/yargs/countEntriesInDirectory) CLI example...
Retargeted to `main`, will check this as soon as I can - have been slammed with some other priorities recently ❤️
Additionally, we'd want to add a CI file for this project before merging 👍🏻
> The /upload assumes there is an uploads folder, and if it is missing, it throws. > There are two ways to solve this: > > * document this >...
On pointers for CI, I'd say start with the current CI workflow that we have - you basically just need to adapt it: https://github.com/nodejs/examples/blob/main/.github/workflows/cli-yargs-countentriesindirectory.yaml There's also the official reference and...
Seems that tap isn't getting installed which is failing the build. If you could figure out why, that'd be awesome :)