Skip to content
This repository has been archived by the owner on Oct 11, 2021. It is now read-only.

Adhere to the maximum line length for YAML adopted by CFN #38

Closed
villasv opened this issue Jan 4, 2018 · 3 comments
Closed

Adhere to the maximum line length for YAML adopted by CFN #38

villasv opened this issue Jan 4, 2018 · 3 comments

Comments

@villasv
Copy link
Owner

villasv commented Jan 4, 2018

Uploading the code to the designer and then downloading it will convert a vew string blocks from standard line continuation | to >, introducing a bunch of blank lines. This is a minor inconvenience, the changes are rapidly undone.

A very minor issue, but making the upload/download idempotent would be pretty convenient.

@villasv villasv added this to Doing in Turbine Board Jan 4, 2018
@villasv
Copy link
Owner Author

villasv commented Jan 5, 2018

Particularly complicated for sections about files instead of commands. Airflow celery config vars are almost already 50 chars long... abuse Fn:Sub?

@villasv
Copy link
Owner Author

villasv commented Mar 7, 2018

Will also be facilitated by #34 if we remove those long scripts from the YAML file

@villasv villasv moved this from Doing to Todo in Turbine Board Nov 14, 2018
@villasv villasv added this to the Blazing Blades milestone Nov 22, 2018
@villasv villasv moved this from Backlog to Doing in Turbine Board Nov 30, 2018
@villasv
Copy link
Owner Author

villasv commented Dec 4, 2018

Not completely done, but it's pretty consistent now except for the environment variables section.

@villasv villasv closed this as completed Dec 4, 2018
Turbine Board automation moved this from Doing to Done Dec 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Turbine Board
  
Done
Development

No branches or pull requests

1 participant