Closed Bug 1460012 Opened 6 years ago Closed 5 years ago

Update taskcluster-cli for r14y

Categories

(Taskcluster :: Services, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1560650

People

(Reporter: bstack, Assigned: pmoore)

References

Details

Explained in bug 1428417.
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.
Oh, I've just seen, this bug is assigned to me! LOL, so I guess I'll be doing it! xD
++ to removing the code-gen (and maybe bug 1444458?) in the process!
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.
https://github.com/taskcluster/taskcluster-cli/pull/192 fixes this for signin, but doesn't touch the rest of the commands.
https://github.com/taskcluster/taskcluster-cli/issues/199 addresses more of the rootUrl bits of this -- but not the code generation stuff.
Component: Redeployability → Services
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.