Closed
Bug 1463456
Opened 7 years ago
Closed 7 years ago
add scopes.md file to beetmoverscript
Categories
(Release Engineering :: Release Automation, enhancement)
Release Engineering
Release Automation
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
Reporter | ||
Comment 1•7 years ago
|
||
Callek is a good resource here for more help and guidance.
Assignee | ||
Comment 2•7 years ago
|
||
Assignee: nobody → lguo
Attachment #8980298 -
Flags: review?(mtabara)
Attachment #8980298 -
Flags: review?(bugspam.Callek)
Reporter | ||
Comment 3•7 years ago
|
||
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.
Reporter | ||
Comment 4•7 years ago
|
||
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+
Reporter | ||
Comment 5•7 years ago
|
||
Thanks for adding this Lisa!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•3 months ago
|
Component: Release Automation: Uploading → Release Automation
You need to log in
before you can comment on or make changes to this bug.
Description
•