Minor API Key Readme Update

[sdhoyt]

Change Summary

Overview

In the documentation for setting the API Key, I moved the environment variable setting step to a line before using the imap-data-access, so it's more clear that it's not part of the imap-data-access command. I also added export for setting the environment variable

[sdhoyt]

pre-commit.ci autofix

[greglucas]

I think that @tech3371 didn't want the export listed as a recommendation because then it sticks around for people in their environment and we don't want people accidentally uploading things in the future because they were still authenticated.

I'm fine with this though.

[sdhoyt]

I think that @tech3371 didn't want the export listed as a recommendation because then it sticks around for people in their environment and we don't want people accidentally uploading things in the future because they were still authenticated.

I'm fine with this though.

@greglucas I'll remove the export. For some reason it didn't seem to work for me without it, but maybe I mistyped something

[sdhoyt]

pre-commit.ci autofix

[greglucas]

It won't work like this, I agree. So we should either revert this or keep what you had before. The export sets it for the entire session of the terminal, so it does work across lines. Whereas this now sets it on one line, then the next line forgets it. Previously this was all on the same line so that the API_KEY would affect the entire line but only that one line/command and not subsequent ones. Does that make sense?