Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove legacy es5 bundling resources #204

Closed
blackfalcon opened this issue Apr 24, 2023 · 1 comment · Fixed by #210
Closed

Remove legacy es5 bundling resources #204

blackfalcon opened this issue Apr 24, 2023 · 1 comment · Fixed by #210
Assignees
Labels
auro-button Type: Perf Performance update to existing code

Comments

@blackfalcon
Copy link
Member

Is your feature request related to a problem? Please describe.

This repo still contains resources related to the bundling of an es5 module, local demo Sass, and still supports legacy concepts like non-rem CSS post-processing.

Describe the solution you'd like

Remove all es5 resources required for bundling. This includes all script references and npm dependencies.

Remove these legacy services. Also, make sure that they are removed from all documentation including the README and API docs.

Update the index.html file in the ./dir to use CDN style resources. The local demo Sass resources should be deleted. The npm script for processing these demo styles should also be released

Please delete any _legacy directory if one exists.

Exit criteria

This issue will be complete once all legacy Sass and es5 building resources have been removed from the repo.

See linked commits for update references.

This issue was copied from #61 by @blackfalcon.

This issue was copied from #56 by @blackfalcon.

@blackfalcon blackfalcon added the Type: Perf Performance update to existing code label Apr 24, 2023
@blackfalcon blackfalcon added this to the Node 18+ update milestone Apr 24, 2023
@blackfalcon blackfalcon transferred this issue from AlaskaAirlines/auro-flight Apr 24, 2023
@blackfalcon blackfalcon self-assigned this Apr 24, 2023
blackfalcon added a commit that referenced this issue Apr 24, 2023
BREAKING CHANGE: This commit removes all legacy es6 build process
including light-dom support, fixed CSS, non CSS variable support,
and es5 bundling.
@blackfalcon blackfalcon linked a pull request Apr 24, 2023 that will close this issue
6 tasks
blackfalcon added a commit that referenced this issue Apr 25, 2023
BREAKING CHANGE: This commit removes all legacy es6 build process
including light-dom support, fixed CSS, non CSS variable support,
and es5 bundling.

Changes to be committed:
modified:   README.md
deleted:    babel.config.js
deleted:    demo/alert.js
modified:   demo/demo.md
modified:   demo/index.html
deleted:    demo/sass/style.scss
deleted:    index.html
modified:   package-lock.json
modified:   package.json
modified:   packageScripts/postinstall.js
modified:   rollup.config.js
modified:   scripts/postCss.js
deleted:    scripts/removeNonRemPlugin.js
deleted:    src/auro-button-light.js
modified:   src/auro-button.js
deleted:    src/es5.js
deleted:    src/style-fixed.scss
deleted:    src/style-ld.scss
@blackfalcon blackfalcon added the Status: Work In Progress Issue or Pull Request work is in Progress label Apr 25, 2023
@blackfalcon
Copy link
Member Author

See commit for review e10e513

@blackfalcon blackfalcon removed the Status: Work In Progress Issue or Pull Request work is in Progress label Apr 28, 2023
blackfalcon added a commit that referenced this issue Jun 1, 2023
BREAKING CHANGE: This commit removes all legacy es6 build process
including light-dom support, fixed CSS, non CSS variable support,
and es5 bundling.

Changes to be committed:
modified:   README.md
deleted:    babel.config.js
deleted:    demo/alert.js
modified:   demo/demo.md
modified:   demo/index.html
deleted:    demo/sass/style.scss
deleted:    index.html
modified:   package-lock.json
modified:   package.json
modified:   packageScripts/postinstall.js
modified:   rollup.config.js
modified:   scripts/postCss.js
deleted:    scripts/removeNonRemPlugin.js
deleted:    src/auro-button-light.js
modified:   src/auro-button.js
deleted:    src/es5.js
deleted:    src/style-fixed.scss
deleted:    src/style-ld.scss
blackfalcon added a commit that referenced this issue Jun 6, 2023
BREAKING CHANGE: This commit removes all legacy es6 build process
including light-dom support, fixed CSS, non CSS variable support,
and es5 bundling.

Changes to be committed:
modified:   README.md
deleted:    babel.config.js
deleted:    demo/alert.js
modified:   demo/demo.md
modified:   demo/index.html
deleted:    demo/sass/style.scss
deleted:    index.html
modified:   package-lock.json
modified:   package.json
modified:   packageScripts/postinstall.js
modified:   rollup.config.js
modified:   scripts/postCss.js
deleted:    scripts/removeNonRemPlugin.js
deleted:    src/auro-button-light.js
modified:   src/auro-button.js
deleted:    src/es5.js
deleted:    src/style-fixed.scss
deleted:    src/style-ld.scss
AuroDesignSystem pushed a commit that referenced this issue Jun 6, 2023
# [7.0.0](v6.6.0...v7.0.0) (2023-06-06)

### Code Refactoring

* **es5:** remove legacy features and build support [#204](#204) ([c0348c4](c0348c4))
* **id:** remove internal ID support [#130](#130) ([1d8d16c](1d8d16c))

### Features

* add multiple updates [#206](#206) ([449c0c4](449c0c4))
* **enum:** add variants support [#160](#160) ([05509b8](05509b8))
* **ondark:** update background values [#156](#156) ([b6ef5f8](b6ef5f8))

### Performance Improvements

* **focus-visible:** remove support for polyfill [#208](#208) ([5f0f8d1](5f0f8d1))
* **icon:** remove internal icon placement support [#209](#209) ([eec81f6](eec81f6))

### BREAKING CHANGES

* This commit includes an update to the
npm namespace to @AuroDesignSystem.

Changes to be committed:
new file:   .github/CODEOWNERS
deleted:    .github/ISSUE_TEMPLATE/audit.md
deleted:    .github/ISSUE_TEMPLATE/bug_report.md
new file:   .github/ISSUE_TEMPLATE/bug_report.yml
deleted:    .github/ISSUE_TEMPLATE/develop-design-story.md
deleted:    .github/ISSUE_TEMPLATE/feature_request.md
new file:   .github/ISSUE_TEMPLATE/feature_request.yml
deleted:    .github/ISSUE_TEMPLATE/general-support.md
new file:   .github/ISSUE_TEMPLATE/general-support.yml
deleted:    .github/ISSUE_TEMPLATE/new_wc.md
deleted:    .github/ISSUE_TEMPLATE/parent-design-story.md
deleted:    .github/ISSUE_TEMPLATE/review-design-story.md
deleted:    .github/ISSUE_TEMPLATE/symbol-design-story.md
deleted:    .github/ISSUE_TEMPLATE/user-story.md
modified:   .github/settings.yml
modified:   .github/workflows/testPublish.yml
modified:   .gitignore
new file:   .husky/pre-commit
modified:   .npmignore
modified:   index.js
deleted:    karma.conf.js
modified:   package-lock.json
* **icon:** This commit will remove all legacy support
for placement of icons within the element other than supporting
icons as slotted content.

Changes to be committed:
modified:   docs/api.md
modified:   src/auro-button.js
modified:   src/style.scss
* **id:** This commit will remove all support for
ID attribute reflection from the parent element to the inner
button element.

Changes to be committed:
modified:   src/auro-button.js
modified:   test/auro-button.test.js
modified:   packageScripts/postinstall.js
* **es5:** This commit removes all legacy es6 build process
including light-dom support, fixed CSS, non CSS variable support,
and es5 bundling.

Changes to be committed:
modified:   README.md
deleted:    babel.config.js
deleted:    demo/alert.js
modified:   demo/demo.md
modified:   demo/index.html
deleted:    demo/sass/style.scss
deleted:    index.html
modified:   package-lock.json
modified:   package.json
modified:   packageScripts/postinstall.js
modified:   rollup.config.js
modified:   scripts/postCss.js
deleted:    scripts/removeNonRemPlugin.js
deleted:    src/auro-button-light.js
modified:   src/auro-button.js
deleted:    src/es5.js
deleted:    src/style-fixed.scss
deleted:    src/style-ld.scss
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auro-button Type: Perf Performance update to existing code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant