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

New core repository #6950

Merged
merged 50 commits into from
Sep 8, 2023
Merged

New core repository #6950

merged 50 commits into from
Sep 8, 2023

Conversation

sethidden
Copy link
Contributor

@sethidden sethidden commented Sep 1, 2023

This PR introduces the following major changes:

Minor/secondary changes:

  • start using unified ci/cd for all packages; align containing code with the integrations team eslint config etc.
  • setup sonarcloud for all 3 packages

internal ticket: IN-3345

@gitguardian
Copy link

gitguardian bot commented Sep 1, 2023

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id Secret Commit Filename
7561600 Company Email Password dabf894 packages/core/tests/factories/useUserFactory.spec.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@sethidden sethidden marked this pull request as ready for review September 7, 2023 06:54
@WojtekTheWebDev WojtekTheWebDev requested a review from a team September 7, 2023 07:46
@WojtekTheWebDev WojtekTheWebDev changed the title IN-3345 new core repository New core repository Sep 8, 2023
@sonarcloud
Copy link

sonarcloud bot commented Sep 8, 2023

[vuestorefront_vue-storefront_sdk] SonarCloud Quality Gate failed.    Quality Gate failed

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

29.4% 29.4% Coverage
4.2% 4.2% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@sonarcloud
Copy link

sonarcloud bot commented Sep 8, 2023

[vuestorefront_vue-storefront_cli] SonarCloud Quality Gate failed.    Quality Gate failed

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

19.0% 19.0% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@sonarcloud
Copy link

sonarcloud bot commented Sep 8, 2023

[vuestorefront_vue-storefront_middleware] 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 0 Code Smells

97.2% 97.2% Coverage
0.0% 0.0% Duplication

@sethidden sethidden merged commit 57b13da into main Sep 8, 2023
13 of 19 checks passed
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.

2 participants