verify format of desktop releases' changesets file

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: bhearsum, Assigned: bhearsum)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [shipit])

Attachments

(2 attachments)

We do this for the mobile JSON changesets, but don't have a similar check for Firefox
Created attachment 696337 [details] [diff] [review]
separate plain text changesets parsing into a function

With tests!
Attachment #696337 - Flags: review?(catlee)
Created attachment 696338 [details] [diff] [review]
verify changesets upon submission
Attachment #696338 - Flags: review?(catlee)
Attachment #696337 - Flags: review?(catlee) → review+
Attachment #696338 - Flags: review?(catlee) → review+
Landed in http://git.mozilla.org/?p=build/release-kickoff.git;a=commit;h=2f372f772acf6737691b96502fb797fdc8584bd3.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Attachment #696338 - Flags: checked-in+
Attachment #696337 - Flags: checked-in+
Whiteboard: [kickoff] → [shipit]
Product: mozilla.org → Release Engineering
Comment on attachment 696337 [details] [diff] [review]
separate plain text changesets parsing into a function

Review of attachment 696337 [details] [diff] [review]:
-----------------------------------------------------------------

Just found a (very small) bug in this patch...

Pete

::: lib/python/release/l10n.py
@@ +61,3 @@
>              if revision == 'FIXME':
>                  raise Exception('Found FIXME in %s for locale "%s"' % \
>                                  (fileName, locale))

The variable "fileName" (above) no longer exists, because the function definition changed from:

getL10nRepositories(fileName, l10nRepoPath, relbranch=None)
to
getL10nRepositories(changesets, l10nRepoPath, relbranch=None)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Opened bug 937175 for this issue instead.
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.