Distribution, expand the ignored distributios set

[ValarDragon]

Expands the set of gauges we ignore, as we are now no longer concerned about state compat in main.

This changes

if remainCoins.Len() == 1 && remainCoins[0].Amount.LTE(osmomath.NewInt(10)) && gauge.DistributeTo.Denom == "gamm/pool/1" && remainCoins[0].Denom != "uosmo" {

to

	// Remove some spam gauges that are not worth distributing. (We ignore the denom stake because of tests.)
	if remainCoins.Len() == 1 && remainCoins[0].Amount.LTE(osmomath.NewInt(100)) && remainCoins[0].Denom != "stake" {
		ctx.Logger().Debug(fmt.Sprintf("gauge debug, this gauge is perceived spam, skipping %d", gauge.Id))
		err := k.updateGaugePostDistribute(ctx, gauge, totalDistrCoins)
		return true, totalDistrCoins, err
	}
	return false, totalDistrCoins, nil
}

Requiring a gauge to distribute at least 100 coins, for any pool. Ignores "stake" to avoid fixing all the tests.

[czarcas7ic]

(We ignore the denom stake because of tests.)

LOL, cant we just inc the tests to be greater than 100??

[ValarDragon]

yeah but its pain

[mattverse]

I think it'd be worth creating unit test for this as well

[czarcas7ic]

Added 5cf383a

[github-actions]

This pull request has been automatically marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. Thank you!