Fix v2 aws sync commands

[chrisjkuch]

cookiecutter-data-science/{{ cookiecutter.repo_name }}/Makefile

Line 62 in b4c0c12

aws s3 sync s3://{{ cookiecutter.dataset_storage.s3.bucket }}/data/ data/\

In v2, sync_data_up incorrectly lists the bucket as the source directory, rather than the local data/ folder. The order should be filpped.

In addition, there is inconsistent use of environment variables. sync_data_down uses the templatized AWS profile from the cookiecutter form:

cookiecutter-data-science/{{ cookiecutter.repo_name }}/Makefile

Line 51 in b4c0c12

data/ {% if cookiecutter.dataset_storage.s3.aws_profile != 'default' %} --profile {{ cookiecutter.dataset_storage.s3.aws_profile }}{% endif %}

while sync_data_up uses the PROFILE environment variable which is not set in the Makefile.

cookiecutter-data-science/{{ cookiecutter.repo_name }}/Makefile

Line 63 in b4c0c12

{% if cookiecutter.dataset_storage.s3.aws_profile %} --profile $(PROFILE){% endif %}