Renames AccountsPackageType to AccountsPackageKind

[brooksprumo]

Problem

When creating an enum to select some underlying kind of a thing, we sometimes append Enum or Type for the name of the type. I think both of these are bad, but not life ending.

Obviously the variants are not types in the Rust sense of the word. And the type is already an enum, so adding Enum feels redundant.

In the Rust std lib, they have this problem for errors. They've solved this with ErrorKind.

(Some non-Rust projects have also used Flavor too, which I like, but not as much as Kind.)

AccountsPackageType has this naming problem.

Summary of Changes

Rename AccountsPackageType to AccountsPackageKind.

[codecov]

Codecov Report

Merging #32908 (ddd886d) into master (4d3bcc5) will increase coverage by 0.0%.
Report is 4 commits behind head on master.
The diff coverage is 91.2%.

❗ Current head ddd886d differs from pull request most recent head 130cb28. Consider uploading reports for the commit 130cb28 to get more accurate results

@@           Coverage Diff           @@
##           master   #32908   +/-   ##
=======================================
  Coverage    82.0%    82.0%           
=======================================
  Files         785      785           
  Lines      212377   212375    -2     
=======================================
+ Hits       174171   174208   +37     
+ Misses      38206    38167   -39     

[apfitzge]

lgtm

[KirillLykov]

Thanks for the explanation, I was thinking about Type vs Kind recently and wonder what is the best-practice way.