New credentials

[tonisives]

HMKit can be intialized with either HMKitOAuthCredentials or HMKitPrivateKeyCredentials.

HMKitOAuthCredentials:

    HMKitCredentials credentials = new HMKitOAuthCredentials(
      "client_id",
      "client_secret"
    );

    HMKitConfiguration configuration = new HMKitConfiguration.Builder()
      .credentials(credentials)
      .environment(HMKitFleet.Environment.SANDBOX)
      .build();

    HMKitFleet hmkit = new HMKitFleet(configuration);

HMKitPrivateKeyCredentials

    HMKitCredentials credentials = new HMKitPrivateKeyCredentials(
      "client_id",
      "private_key_json_file_contents"
    );

    HMKitConfiguration configuration = new HMKitConfiguration.Builder()
      .credentials(credentials)
      .environment(HMKitFleet.Environment.SANDBOX)
      .build();

    HMKitFleet hmkit = new HMKitFleet(configuration);

[codecov]

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

[slashmili]

@tonisives what do you think in case of private key credentials, we actually ask for private key and private key id?

    HMKitCredentials credentials = new HMKitPrivateKeyCredentials(
      "client_id",
      "private_key_id",
      "private_key_pem"
    );

[tonisives]

@tonisives what do you think in case of private key credentials, we actually ask for private key and private key id?

    HMKitCredentials credentials = new HMKitPrivateKeyCredentials(
      "client_id",
      "private_key_id",
      "private_key_pem"
    );

It might be easier for users to copy/paste the json file contents.
We had a problem where users didn't include the -----BEGIN PRIVATE KEY-----\n part.

But negative is that if we change the json file keys/content then the parsing in hmkit-fleet fails

Either option is fine with me

[slashmili]

I understand your concern. However I'd say people make mistake and then learn(or read the doc) and use it properly :D

Let's go for explicit configuration and only pass what is needed.

[slashmili]

LGTM 🚀