Add initial dev setup automation #231
Conversation
| .PHONY: shell | ||
| shell: | ||
| @export CONDA_ENV_PROMPT='<{name}>' | ||
| @echo 'conda activate $(env)' |
There was a problem hiding this comment.
I have been unable to coerce this into working in other repos. As-is, this is a no-op echo.
There was a problem hiding this comment.
I'm not sure if you are + or - on this one :D
This specific cmd is not that relevant anymore(and I think I'll remove it) no that we want to default using conda run... right? Or am I missing something?
There was a problem hiding this comment.
Yeah, I was indirectly suggesting we remove it, unless you have a way to make it work :)
pyscriptjs/Makefile
Outdated
| .PHONY: setup | ||
| setup: | ||
| @if [ -z "$${CONDA_SHLVL:+x}" ]; then echo "Conda is not installed." && exit 1; fi | ||
| $(CONDA_EXE) env $(shell [ -d $(env) ] && echo update || echo create) -p $(env) --file environmentwh.yml |
There was a problem hiding this comment.
Is this environment filename a typo?
There was a problem hiding this comment.
Yeah.. doing a 100 thing at the same time type of typo. Thanks for the catch!
Co-authored-by: Matt Kramer <matthew.robert.kramer@gmail.com>
Co-authored-by: Matt Kramer <matthew.robert.kramer@gmail.com>
…t into fpliger/69_add_makefile
.pre-commit-config.yaml
Outdated
| # This is the configuration for pre-commit, a local framework for managing pre-commit hooks | ||
| # Check out the docs at: https://pre-commit.com/ | ||
|
|
||
| repos: [] |
There was a problem hiding this comment.
pre-commit will add a newline here at the end of the file :) Not important right now.
Co-authored-by: Matt Kramer <matthew.robert.kramer@gmail.com>
Addresses #69