search

unitedstates / congressional-record

A parser for the Congressional Record.
Other
128 stars 40 forks source link

Rename main parser file? #9

Closed konklone closed 10 years ago

konklone commented 10 years ago

How about parse instead of parser.py?

drinks commented 10 years ago

Verb module names weird me out a little bit, but I could see a parse() function living in parser.py that gets called by __main__, is that reasonable?

konklone commented 10 years ago

I'm only thinking about the CLI experience, not the internal layout. I think of parser.py as a script, not a module, but I don't have strong feelings on the matter, so feel free to close it if you want.

drinks commented 10 years ago

Ok, that's fair. If it adds no value to the import-and-run-programmatically experience such that there's no need to use it as a starting point from code, then :thumbsup: