Dan Lynch

Results 169 comments of Dan Lynch

so the PR is now green - could close this if merged :)

this would require a major version bump!

@ethanresnick this is super cool! I've been maintaining another fork of the parser that uses this library. I'd love to use your library and will start to integrate it :)

@ethanresnick I just upgraded this, and tested it out, and it's way more portable! Worked on node 10, 12, and 14! Thanks for putting this together. Hope to stay in...

To reduce docker size, I attempted to remove a few `RUN` commands to help reduce layers. As far as de-duplication via a base image, I'm not sure how to properly...

Other than splitting out into separate parts for de-duplication, were there other issues with it? I wasn't sure how to manage your CI portion, happy to adjust things to work...

@theory I totally understand. What I'm suggesting is that there really isn't much in common, because there are non-common flags scattered from top to bottom that cannot be moved based...

Well, they are working as is, but I notice that the CI/CD is not passing. I imagine steps left are - get CI/CD passing - automate the build/push process

I think the issue is that some of the errors aren't getting caught in the right place for the app. For me authentication is an easy redirect we can handle,...