chore: temporary disable tree-shaking #11586

[ncoden]

Description

Despite the previous attempts to make Foundation compatible with tree shaking (#11445, #11497 and #11508), Foundation is used with side-effects (on jQuery) and cannot be considered as "without side-effects" for tree-shaking purposes.

For example:

import $ from 'jquery';
import Foundation from 'foundation-sites'; // may not be included

$(document).foundation();

This pull request removes the "sideEffects: false" label and thus disable tree-shaking support, until the Foundation API and usage is changed to not implicitely rely on the global jQuery object.

Closes #11586

Related to:

• #11508 - chore: provide CJS, ESM and ES6+ bundles
• #11497 - docs: use named imports in javascript documentation
• #11445 - chore: improve "tree shaking" support and documentation

Types of changes

• Documentation
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (anything that would change an existing functionality)
• Maintenance (refactor, code cleaning, development tools...)

Checklist

• I have read and follow the CONTRIBUTING.md document.
• The pull request title and template are correctly filled.
• The pull request targets the right branch (develop or develop-v...).
• My commits are correctly titled and contain all relevant information.
• I have updated the documentation accordingly to my changes (if relevant).
• I have added tests to cover my changes (if relevant).