Brian J. Cardiff

Results 130 comments of Brian J. Cardiff

It’s still something that should be fixed. I will leave this open until then. Thanks for reporting it.

What about leaving the binary in `./bin/app`? I think that could be a better change because for new crystal apps the shard.yml will include [a targets description](https://github.com/crystal-lang/shards/blob/master/SPEC.md#targets) (and we could...

There was an idea of having a special branch for shards metadata in the past. A place where the requirements of all published versions can be available at once without...

To me the fact that installing executables is done after a postinstall does not imply that is part of it. If it would, then it should be inside the `postinstall:`...

The master of this repo requires Crystal 0.36.0 since #72 was merged. Either update to latest crystal or use `ref: 513e51e0eea9341cd095c820a4563bbf6a53a2e5` (last commit with 0.35 support) in your shard.yml declaration...

That information could be nice for humans in the shard.lock, yes. But since molinillo didn't need we avoid adding it. In a Gemfile lock you can see some of that...

[DB.open without a block](http://crystal-lang.github.io/crystal-db/api/0.5.0/DB.html#open%28uri%3AURI%7CString%29-class-method) will give you a Database instance . Even with block syntax you are allowed to have multiple Database instances.

That's right, shards looks for `v{version}` tags only as convention.

@kostya that might change the indentation and blanks a lot. Isn't the yaml parser capable of tell the location of the different nodes so the edit can be expressed in...

I like ``` $ crystal|shards add NAME $ crystal|shards add NAME --dev ``` To modify shards.yml. --dev to be a development dependency. I wouldn't run the installer right away. NAME...