Closed Bug 1463456 Opened 7 years ago Closed 7 years ago

add scopes.md file to beetmoverscript

Categories

(Release Engineering :: Release Automation, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mtabara, Assigned: lguo)

References

Details

Attachments

(1 file)

In bug 1449508 we reached a consensus to have descriptive names for our scopes. One of the follow-ups is to improve the docs and specify a `scopes.md` in each of the *scripts. The work for this has been done in all scripts, except beetmover. We need to do it there as well. More examples: * addonscript[1] * pushapkscript[2] * treecript[3] * signingscript[4] * balrogscript[5] ... In-tree reference where most of these scopes are defined lies within [6]. [1]: https://github.com/mozilla-releng/addonscript/pull/55 [2]: https://github.com/mozilla-releng/pushapkscript/pull/40/ [3]: https://github.com/mozilla-releng/treescript/pull/22 [4]: https://github.com/mozilla-releng/signingscript/pull/51 [5]: https://github.com/mozilla-releng/balrogscript/pull/35 [6]: https://hg.mozilla.org/mozilla-central/file/tip/taskcluster/taskgraph/util/scriptworker.py
Callek is a good resource here for more help and guidance.
Assignee: nobody → lguo
Attachment #8980298 - Flags: review?(mtabara)
Attachment #8980298 - Flags: review?(bugspam.Callek)
Comment on attachment 8980298 [details] [review] added SCOPES.md to beetmoverscript Thanks for adding this! Looks good but we still need to amend some of the things a bit before we can merge. Dropped a few comments in the PR.
Comment on attachment 8980298 [details] [review] added SCOPES.md to beetmoverscript R+'ed after refactoring. Confirmed on IRC that both me and Callek are happy with this. https://github.com/mozilla-releng/beetmoverscript/commit/6c82e0f3da436cd15755dc383b5aee18ba1785d7 This is going to be picked by the next beetmoverscript bump.
Attachment #8980298 - Flags: review?(mtabara)
Attachment #8980298 - Flags: review?(bugspam.Callek)
Attachment #8980298 - Flags: review+
Thanks for adding this Lisa!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: Release Automation: Uploading → Release Automation
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: