deprecate incydr functionality

[ceciliastevens]

Marks functionality that is superseded by the Incydr SDK (https://developer.code42.com/) as deprecated.

[tora-kozic]

Have we tested this alongside the cli update? Do the py42 deprecations carry over to that?

If we add a deprecation warning to the client itself, does every command also print that out (ie. if we add it to the client and the command does it do it twice?)

[ceciliastevens]

Have we tested this alongside the cli update? Do the py42 deprecations carry over to that?

If we add a deprecation warning to the client itself, does every command also print that out (ie. if we add it to the client and the command does it do it twice?)

Yes, the code42cli prints out warnings generated by its dependencies, including py42, so if we make these changes together both the SDK's warning and the CLI's warning will print (as do warnings from other dependencies, currently it looks like there's one that pandas is throwing).

[ceciliastevens]

Have we tested this alongside the cli update? Do the py42 deprecations carry over to that?
If we add a deprecation warning to the client itself, does every command also print that out (ie. if we add it to the client and the command does it do it twice?)

Yes, the code42cli prints out warnings generated by its dependencies, including py42, so if we make these changes together both the SDK's warning and the CLI's warning will print (as do warnings from other dependencies, currently it looks like there's one that pandas is throwing).

I actually spoke too soon on this one. It was a specific choice in how I was doing the warnings for the alert queries that was causing this. I updated how that is done and now it's only warning once.