Skip to content

Add explicit order hints to YAML SQL definitions - #8

Open
wolfsage wants to merge 1 commit into
fastmail:masterfrom
wolfsage:fix-warnings
Open

Add explicit order hints to YAML SQL definitions#8
wolfsage wants to merge 1 commit into
fastmail:masterfrom
wolfsage:fix-warnings

Conversation

@wolfsage

@wolfsage wolfsage commented Jun 5, 2017

Copy link
Copy Markdown
Contributor

This shuts up SQL::Translator::Parser::YAML warnings...

This shuts up SQL::Translator::Parser::YAML warnings...
@rjbs

rjbs commented Jun 13, 2017

Copy link
Copy Markdown
Member

I look at this and think two things:

  1. thank goodness to have an end to warnings
  2. I am going to screw this up when I change something later

Can we use a sequence instead of a mapping for these fields, so they're naturally ordered?

@wolfsage

Copy link
Copy Markdown
Contributor Author

Hrm, how do you envision the sequence looking?

Something that's interpolated and increments?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants