Thomas Sibley
Thomas Sibley
This is still an issue on: ``` Terraform v1.3.1 on linux_amd64 + provider registry.terraform.io/hashicorp/aws v4.32.0 ```
Hmm, ok. Out of publicly available code, I did find [MOOSE](https://github.com/idaholab/moose) [using it](https://cs.github.com/idaholab/moose?q=pin_depends).
> Could it be that we're not packaging it correctly? Probably not, this is just how NodeJS rolls. Huge `node_modules/` are a widely-recognized issue. For reference, my local `node_modules/` for...
It would be nice to do, for sure.
This would be a great improvement to the error message.
Making this adjustable makes sense to me. There's a few places in Augur and Auspice which special-case the `clade_membership` field. We'd potentially want to make the special-cased behaviour also apply...
I'd vote for (2), which was [a previous suggestion of mine as well](https://github.com/nextstrain/.github/issues/21).
The validation errors are unfortunately poor here because of bad support for the JSON Schema `oneOf` type in the Python library we're using to validate. The actual errors don't percolate...
@corneliusroemer I broadly agree. The validation aspect of `augur export` is well-intentioned but I think poor (and here, actively bad) in practice. It could be better and actually helpful, but...
`--mask-gaps if-only-gaps terminal` to me reads as the `terminal` option conditioned by `if-only-gaps` or something. It doesn't parse to me as the union of their effects (e.g. always mask terminal...