search

fastmail / Moonpig

the moonpig billing system
69 stars 12 forks source link

Add explicit order hints to YAML SQL definitions #8

Open wolfsage opened 7 years ago

wolfsage commented 7 years ago

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

rjbs commented 7 years ago

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 commented 7 years ago

Hrm, how do you envision the sequence looking?

Something that's interpolated and increments?