PR#37: Refactor code to re-organize handlers v2 -

freshmaker

#37 Refactor code to re-organize handlers v2

Closed May 31, 2017 by qwan. Opened May 27, 2017 by qwan.

qwan/freshmaker refactor-handlers-v2 into master

qwan commented May 27, 2017

Changes:

Parsers are organized with dirs for sources (git, mbs, koji, bodhi).
Events, parsers and handlers are generally renamed to include message
topic. One exception is those for topic of git.receive which is splitted
into multiple events, parsers and handlers.
Handlers for building containers are placed under
handlers/containers, and handlers for building modules are placed under
handlers/modules.
Have one handler for each event, this is also why naming the handlers
with message topic included.
Updated unittest cases per the changes.

qwan commented May 27, 2017

Uploaded another version for re-organizing the handlers, compare to the version in #35, in this version, handlers under handlers/modules are responsible for building modules, and handlers under handlers/containers are responsible for building containers. While in #35, handlers are organized by dirs of event sources.

jkaluza commented May 29, 2017

Commented on #35.

jkaluza commented May 29, 2017

I've renamed the title to "v2" to make it clear it is not the same PR somehow submitted twice.

qwan commented May 31, 2017

Abandon this with #35 merged.

Pull-Request has been closed by qwan

May 31, 2017