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

Update deriv-charts to 2.5.0 #16651

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

DerivFE
Copy link
Collaborator

@DerivFE DerivFE commented Aug 28, 2024

This PR updates @deriv/deriv-charts to 2.5.0

Copy link

vercel bot commented Aug 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Aug 28, 2024 9:46am

Copy link

@review-deriv review-deriv left a comment

Choose a reason for hiding this comment

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

PR description is too short. You can use this format:

# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.

# Cards
* https://clickup.com/12345
* https://clickup.com/12346

Copy link
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 4 package(s) with unknown licenses.
See the Details below.

License Issues

package-lock.json

PackageVersionLicenseIssue Type
@deriv/deriv-charts2.5.0NullUnknown License

packages/bot-web-ui/package.json

PackageVersionLicenseIssue Type
@deriv/deriv-charts^2.5.0NullUnknown License

packages/core/package.json

PackageVersionLicenseIssue Type
@deriv/deriv-charts^2.5.0NullUnknown License

packages/trader/package.json

PackageVersionLicenseIssue Type
@deriv/deriv-charts^2.5.0NullUnknown License

OpenSSF Scorecard

Scorecard details
PackageVersionScoreDetails
npm/@deriv/deriv-charts 2.5.0 UnknownUnknown
npm/@deriv/deriv-charts 2.4.0 UnknownUnknown
npm/dotenv 16.4.5 🟢 3.5
Details
CheckScoreReason
Code-Review⚠️ 1Found 3/16 approved changesets -- score normalized to 1
Maintained🟢 103 commit(s) and 10 issue activity found in the last 90 days -- score normalized to 10
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Security-Policy⚠️ 0security policy file not detected
Fuzzing⚠️ 0project is not fuzzed
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
Vulnerabilities⚠️ 011 existing vulnerabilities detected
npm/postcss 8.4.41 🟢 4.8
Details
CheckScoreReason
Code-Review🟢 5Found 8/15 approved changesets -- score normalized to 5
Maintained🟢 1030 commit(s) and 5 issue activity found in the last 90 days -- score normalized to 10
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
License🟢 10license file detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Packaging⚠️ -1packaging workflow not detected
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Signed-Releases⚠️ -1no releases found
Security-Policy⚠️ 0security policy file not detected
Fuzzing⚠️ 0project is not fuzzed
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
Vulnerabilities🟢 91 existing vulnerabilities detected
npm/tsconfig-paths 4.2.0 🟢 3.3
Details
CheckScoreReason
Code-Review🟢 5Found 16/30 approved changesets -- score normalized to 5
Maintained⚠️ 00 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: githubv4.Query: Resource not accessible by integration
Security-Policy⚠️ 0security policy file not detected
Fuzzing⚠️ 0project is not fuzzed
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
Vulnerabilities⚠️ 012 existing vulnerabilities detected
npm/typescript-plugin-css-modules 5.1.0 🟢 3.9
Details
CheckScoreReason
Code-Review🟢 4Found 12/26 approved changesets -- score normalized to 4
Maintained⚠️ 00 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: githubv4.Query: Resource not accessible by integration
Packaging⚠️ -1packaging workflow not detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Binary-Artifacts🟢 10no binaries found in the repo
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Security-Policy⚠️ 0security policy file not detected
Fuzzing⚠️ 0project is not fuzzed
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
Vulnerabilities🟢 73 existing vulnerabilities detected
npm/@deriv/deriv-charts ^2.5.0 UnknownUnknown
npm/@deriv/deriv-charts ^2.4.0 UnknownUnknown
npm/@deriv/deriv-charts ^2.5.0 UnknownUnknown
npm/@deriv/deriv-charts ^2.4.0 UnknownUnknown
npm/@deriv/deriv-charts ^2.5.0 UnknownUnknown
npm/@deriv/deriv-charts ^2.4.0 UnknownUnknown

Scanned Manifest Files

package-lock.json
  • @deriv/deriv-charts@2.5.0
  • @deriv/deriv-charts@2.4.0
  • dotenv@16.4.5
  • postcss@8.4.41
  • tsconfig-paths@4.2.0
  • typescript-plugin-css-modules@5.1.0
packages/bot-web-ui/package.json
  • @deriv/deriv-charts@^2.5.0
  • @deriv/deriv-charts@^2.4.0
packages/core/package.json
  • @deriv/deriv-charts@^2.5.0
  • @deriv/deriv-charts@^2.4.0
packages/trader/package.json
  • @deriv/deriv-charts@^2.5.0
  • @deriv/deriv-charts@^2.4.0

Copy link
Contributor

github-actions bot commented Aug 28, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 27
🟧 Accessibility 70
🟢 Best practices 92
🟧 SEO 77
🟧 PWA 78

Lighthouse ran with https://deriv-app-git-chore-update-deriv-charts-to-250.binary.sx/

Copy link

sonarcloud bot commented Aug 28, 2024

Copy link
Contributor

package old new diff pct change
account 191 KB 191 KB 0 B 0% 🟢
appstore 525 KB 525 KB 0 B 0% 🟢
bot-web-ui 784 KB 784 KB +5 B +0.0% 🟢
cashier 614 KB 614 KB 0 B 0% 🟢
cfd 136 KB 136 KB 0 B 0% 🟢
core 6.8 MB 6.8 MB +382 B +0.0% 🟢
p2p 1.0 MB 1.0 MB 0 B 0% 🟢
reports 59 KB 59 KB 0 B 0% 🟢
trader 421 KB 421 KB 0 B 0% 🟢
wallets 3.9 MB 3.9 MB 0 B 0% 🟢

@balakrishna-deriv balakrishna-deriv merged commit 2c6fb94 into master Aug 28, 2024
8 checks passed
@balakrishna-deriv balakrishna-deriv deleted the chore/update-deriv-charts-to-2.5.0 branch August 28, 2024 09:47
@coveralls
Copy link

Coverage Status

coverage: 46.667% (-0.002%) from 46.669%
when pulling e005260 on chore/update-deriv-charts-to-2.5.0
into 4e34e6a on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants