Closed
Bug 1460012
Opened 6 years ago
Closed 5 years ago
Update taskcluster-cli for r14y
Categories
(Taskcluster :: Services, enhancement)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1560650
People
(Reporter: bstack, Assigned: pmoore)
References
Details
Explained in bug 1428417.
Assignee | ||
Comment 2•6 years ago
|
||
Note, taskcluster-cli has no need for code generation, so it might be easier to remove it when it is made redeployable. The benefit of code generation is that it elevates the overstepping of API boundaries by calling code to a compile time error. It is useful when the interface to the user is a language interface, but in the case of a command line utility like taskcluster-cli, the interface is instead the command line arguments passed, which are in any case evaluated at runtime. This should imageine this will simplify the build process, allow the binary to be API-version-independent, and make the job of making it redeployable much easier.
Assignee | ||
Comment 3•6 years ago
|
||
Oh, I've just seen, this bug is assigned to me! LOL, so I guess I'll be doing it! xD
Comment 4•6 years ago
|
||
++ to removing the code-gen (and maybe bug 1444458?) in the process!
Comment 5•6 years ago
|
||
This utility should probably now require that TASKCLUSTER_ROOT_URL be set (or specified as an argument). The `signin` command should also output this value, for completeness.
Comment 6•6 years ago
|
||
https://github.com/taskcluster/taskcluster-cli/pull/192 fixes this for signin, but doesn't touch the rest of the commands.
Comment 7•6 years ago
|
||
https://github.com/taskcluster/taskcluster-cli/issues/199 addresses more of the rootUrl bits of this -- but not the code generation stuff.
Updated•5 years ago
|
Component: Redeployability → Services
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•