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

Fix/multi input is not a function #80

Merged
merged 4 commits into from
Jul 4, 2024

Conversation

alfredosalzillo
Copy link
Owner

Copy link

codecov bot commented Jul 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6fb7530) to head (04fd473).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #80   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           28        28           
  Branches         5         5           
=========================================
  Hits            28        28           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fb7530...04fd473. Read the comment docs.

@alfredosalzillo alfredosalzillo force-pushed the fix/multi-input-is-not-a-function branch 2 times, most recently from fe71e02 to 6d0c809 Compare July 4, 2024 19:05
@alfredosalzillo alfredosalzillo force-pushed the fix/multi-input-is-not-a-function branch from 6d0c809 to 69c822a Compare July 4, 2024 19:20
@alfredosalzillo alfredosalzillo merged commit 91a895c into main Jul 4, 2024
4 checks passed
Copy link

github-actions bot commented Jul 4, 2024

🎉 This PR is included in version 1.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

wridgeu added a commit to wridgeu/chrome-extension-video-playbackrate that referenced this pull request Jul 7, 2024
- now uses UI5 web components 2.0
- uses type imports where necessary
- some getElementById calls are now typed against the UI5 components
- adjusts rollup plugin multiInput usage thanks to the fix in alfredosalzillo/rollup-plugin-multi-input#80
- adjusts rimraf cli usage due to version changes, see also: https://stackoverflow.com/a/75309179/10323879 for a tldr;
- renamed the default checkbox text
- removed obsolete theme support (dev side)
- removed deprecated "install" command from husky
- remove obsolete eslintignore file and move ignored files to main config
- complete move to eslint flatconfig
- file linting and minor code adjustments
- version bumps across the board

comments:
- some of the linting was using biome (as plugin) - I could add biome natively to the project and get rid of eslint, would be nice but not necessary for now
- still manually bumping the versions for now, might add semver later on
- may try out some other bundlers/tools ... what if instead of using raw rollup I could try swc, vite?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants