Simplify the getAccess method in the CLI

[garrettjstevens]

Since we can parse the flags in the base class, this does so to reduce the verbosity of getAccess().

It also makes some fixes to get rid of some eslint-disable directives

[garrettjstevens]

@dariober When I was removing some of the eslint-disable directives, I realized we were using /dev/stdin to read from stdin. Since that's not portable cross-platform (.e.g. on Windows), I changed it to use process.stdin, but had to make the idReader async to do so. I don't think it affected anything, but would appreciate you double-checking.

Also, should this be removed? From what I remembered, .clirc was what we used before we used the conf package.

Apollo3/packages/apollo-cli/src/utils.ts

Line 25 in 28c2d33

const CONFIG_PATH = path.resolve(os.homedir(), '.clirc')

[dariober]

@garrettjstevens Looks good to me

I changed it to use process.stdin, but had to make the idReader async to do so

I changed edit.ts accordingly

Also, should this be removed? From what I remembered, .clirc was what we used before we used the conf package.

Yes, it seems so. I removed it and simplified the checking if user is already logged in (674e4ff).