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(vwc-carousel): white source report high severity for carousel #774

Merged
merged 13 commits into from
Apr 18, 2021

Conversation

yinonov
Copy link
Contributor

@yinonov yinonov commented Apr 14, 2021

swiper major version update breaks scss paths.
as documented in swiper a minimized scss bundle is available
@forward 'swiper/swiper-bundle.min.css';

@gullerya if previous imports deliberately referenced individual components raise a flag

removing @types/swiper as of version 6 swiper is fully typed

@github-actions
Copy link

🚀

Latest successful build of the PR deployed here.

🚀

@yinonov yinonov changed the title fix(vwc-carousel): WhiteSource report high severity for carousel fix(vwc-carousel): white source report high severity for carousel Apr 14, 2021
@tveinfeld
Copy link
Contributor

I mean no harm, let me in

..not by the hair of my chinny, chin, chin!
lol

@yinonov yinonov added the ⚠️ Critical ⚠️ Immediate attention required label Apr 16, 2021
@sonarcloud
Copy link

sonarcloud bot commented Apr 18, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@gullerya gullerya merged commit 3c93585 into master Apr 18, 2021
@gullerya gullerya deleted the VIV-493-swiper-high-severity branch April 18, 2021 13:28
aria-disabled="true"
aria-label="Previous slide"
class="swiper-button-disabled swiper-button-prev swiper-nav"
role="button"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't have been dropped. My bad? I don't recall seeing it in template. Dynamically set in run-time?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ Critical ⚠️ Immediate attention required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants