Skip to content
This repository was archived by the owner on Sep 26, 2018. It is now read-only.

Conversation

@tonybaloney
Copy link
Contributor

Refactored pack to use a consistent formatter and field sets, supports lists and single instances, also supports 1-level deep field mapping to the dictionary. Also added new actions, list_images, list_sizes.

…s lists and single instances, also supports 1-level deep field mapping to the dictionary. Also added new actions, list_images, list_sizes
@manasdk
Copy link
Contributor

manasdk commented Nov 12, 2015

I bet Kami has strong views on this one.

Overall, I see the direction and yeah it does make sense. If we could up with a generic serializer that would be even better but then often it is better to remain pragmatic.

@Kami
Copy link
Member

Kami commented Nov 17, 2015

Thanks and sorry for the delay.

I will look into this today.

@Kami
Copy link
Member

Kami commented Nov 17, 2015

Good change. LGTM +1.

I will merge this and push some minor style changes afterwards.

Kami added a commit that referenced this pull request Nov 17, 2015
Refactored pack to use a consistent formatter and field sets, support…
@Kami Kami merged commit 2051c56 into StackStorm:master Nov 17, 2015
emedvedev pushed a commit that referenced this pull request Apr 4, 2016
Refactored pack to use a consistent formatter and field sets, support…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants