https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py File MoinMoin/PageEditor.py (right): https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py#newcode91 MoinMoin/PageEditor.py:91: @keyword mtime_override: override mtime (default None) please also document what None means. how about just naming it mtime? using current time, if not given would be quite natural. https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py#newcode99 MoinMoin/PageEditor.py:99: self.mtime_override = keywords.get('mtime_override', None) None is the default for .get(), you don't need to give it https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py#newcode1008 MoinMoin/PageEditor.py:1008: if self.mtime_override: ... is not None: https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py#newcode1009 MoinMoin/PageEditor.py:1009: mtime_usecs = self.mtime_override i would not have mtime(_override) as usecs, but rather same unit of measure and type as os.path.getmtime() returns. and then do the wikiutil.timestamp2version() call after the if-block. https://codereview.appspot.com/7153043/diff/1/MoinMoin/PageEditor.py#newcode1018 MoinMoin/PageEditor.py:1018: mtime_usecs = wikiutil.timestamp2version(time.time()) see above. somehow looks duplicated though. https://codereview.appspot.com/7153043/diff/1/MoinMoin/script/import/wikipage.py File MoinMoin/script/import/wikipage.py (right): https://codereview.appspot.com/7153043/diff/1/MoinMoin/script/import/wikipage... MoinMoin/script/import/wikipage.py:44: help='Use TIME (YYYY-MM-DD HH:MM:SS) for edit history / RecentChanges') you should also say what it does if this is not used, then it might get clearer. also, it should be UTC if there is no timezone specified. https://codereview.appspot.com/7153043/diff/1/MoinMoin/script/import/wikipage... MoinMoin/script/import/wikipage.py:79: mtime = timestamp2version(time.mktime(time.strptime(self.options.mtime, "%Y-%m-%d %H:%M:%S"))) be careful, iirc mktime is the inverse operation of localtime, NOT of gmtime. https://codereview.appspot.com/7153043/diff/1/MoinMoin/script/import/wikipage... MoinMoin/script/import/wikipage.py:81: mtime = timestamp2version(time.time()) but this IS utc