airspeed icon indicating copy to clipboard operation
airspeed copied to clipboard

Published 20 hours ago • verified publisher icon purcell

Add 'strict' mode

Open tuukkamustonen opened this issue 4 years ago • 2 comments

Sketched something for https://github.com/purcell/airspeed/issues/39.

  • Using loader as vessel to deliver options. Not perfect, but should work.
  • Should probably be called something else than "strict" mode, to distinguish from Velocity's strict mode. Maybe stricter mode :P ?
  • Needs way more tests, I would say

What do you think? Do you see fundamental problems here?

tuukkamustonen avatar Oct 25 '19 14:10 tuukkamustonen

Thanks! Thinking about this, I guess another way to do it might be to support a well-known object in the namespace, e.g. airspeed_options, or just an airspeed_strict var. Then a user can pass that in as part of the namespace handed to merge. This would be a simpler implementation, and I don't see any particular downside.

purcell avatar Oct 25 '19 23:10 purcell

Yeah, that would work, too. It would kinda feel dirty to pollute user's namespace, but I guess there's no harm so why not. I can change it.

Any other changes you'd like to see here?

tuukkamustonen avatar Oct 28 '19 08:10 tuukkamustonen