Upgrade @blueprintjs/icons to ^3.6.0

[ggallon]

Hi,

This upgrade add new icons : inbox-filtered, inbox-geo, inbox-search, inbox-update, cube, cube-add, cube-remove, lifesaver, table-filtered and clean.

@blueprintjs/icons 3.6.0 : palantir/blueprint#3335 (comment)
@blueprintjs/icons 3.4.0 : palantir/blueprint#3222 (comment)
@blueprintjs/icons 3.3.0 : palantir/blueprint#3052 (comment)

[mshwery]

Looks like size limit is failing with this upgrade.

[ggallon]

Hi @mshwery,

I saw, this upgrade impact the package size of 3.53 KB:

• New size: 207.53 KB
• Size limit: 204 KB

I can increase the size limit to 207.53 or 208 KB?

[mshwery]

@ggallon I would love if there was a way to add tree shaking to the icons in a non-breaking way. I'm hesitant to knowingly increase everyone's bundle size to add a few icons that they don't use today...

We could bump the size limit to 208 for now, but I want to start thinking about ways to actively reduce the bundle size (tree-shaking icons is probably the biggest one).

Potentially another way we could handle it is treat icons as a BYO- bring your own –icons to evergreen... We'd have to provide our own default icons for everything that Evergreen relies on internally (search, close, add, checkmark, etc).

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.