Brian J. Cardiff
Brian J. Cardiff
Maybe this is also needed so the cog generates the new line. ```patch --- cog.out("```\n{}```".format("\n".join(models))) +++ cog.out("```\n{}\n```".format("\n".join(models))) ```
You can also use PGHOST env variable and leave the connection string almost empty. See last spec of conninfo_spec.
For GitHub, from the clone menu we should support - https://github.com/owner/repo.git - [email protected]:owner/repo.git I want to support the most common format the user can copy paste directly. And I want...
Oh and should I move parsing and rendering to a DependencyDefinition type to avoid having these changes in dependency but not used in all places?
I think I incorporated all of the feedback.
The alternative implementation seems good. We can close this PR then in favor of the other. Anything else that is missing?, because it's marked as draft.
Commit pushed into this branch
Made some changes to use `#` instead of `@` while supporting it also in other syntax like `git@` `https://` etc and not just `github:`
Ok, leaving `#` as exact version. (Or I can drop that too). Adding a `version: ` manually is easy, definitely, but this whole thing is to avoid the user editing...
FWIW my intention was/is to take the PR from here. I just haven't pushed my iterations. Meanwhile, it serves for discussing what is missing from POC to something we can...