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

v4 #767

Merged
merged 28 commits into from
Apr 29, 2022
Merged

v4 #767

merged 28 commits into from
Apr 29, 2022

Conversation

chintan9
Copy link
Owner

No description provided.

@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Apr 15, 2022
@guardrails
Copy link

guardrails bot commented Apr 15, 2022

All previously detected findings have been fixed. Good job! 👍🎉

We will keep this comment up-to-date as you go along and notify you of any security issues that we identify.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

height: 100%;
}

body {

Choose a reason for hiding this comment

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

Rule doesn't have all its properties in alphabetical order.

@vizipi
Copy link

vizipi bot commented Apr 15, 2022

Pull request analysis by VIZIPI

Below you will find who is the most qualified team member to review your code.
This analysis includes his/her work on the code included in this Pull request, in addition to their experience in code affected by these changes ( partly found within the list of potential missing files below )   Feedback always welcome

Reviewers with knowledge related to these changes

Match % Person Commit Count Common Files
86.36 % amirHossein-Ebrahimi 36 19

Potential missing files from this Pull request

files commonly committed with a subset of this pr, but not committed this time. (click to collapse)
FilePercentilerate
example/package-lock.json50.00 %1 out of 2 times
src/Plyr.tsx50.00 %1 out of 2 times

Committed file ranks

(click to expand)
  • 99.38%[package.json]
  • 99.69%[package-lock.json]
  • 96.88%[rollup.config.js]
  • 96.56%[tsconfig.json]
  • 86.56%[example/README.md]
  • 99.06%[README.md]
  • 0.00%[.babelrc.json]
  • 97.81%[.github/workflows/nodejs.yml]
  • 20.31%[example/src/vite-env.d.ts]
  • 57.19%[example/src/main.tsx]
  • @sourcelevel-bot
    Copy link

    SourceLevel has finished reviewing this Pull Request and has found:

    • 1 possible new issue (including those that may have been commented here).

    See more details about this review.

    @sonarqubecloud
    Copy link

    Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

    No Coverage information No Coverage information
    0.0% 0.0% Duplication

    @chintan9 chintan9 merged commit 084b59f into master Apr 29, 2022
    @trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Apr 29, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    ✨ Merged Pull Request has been merged successfully size/XXL
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants