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

Adds code standard checking #1009

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adds code standard checking #1009

wants to merge 1 commit into from

Conversation

bradp
Copy link
Contributor

@bradp bradp commented Mar 27, 2024

This is a replacement for #851

This adds the files and configurations for code standards and linting. Another PR based off this branch will include the first round of fixes for it.

  • Adds .editorconfig to make things consistent

  • Adds .phpcs.dist.xml and .php-cs-fixers.dist.php for PHP linting

  • Adds jsconfig.json to allow IDE eslint to correctly associate aliases

  • Updates Composer and NPM commands to include linting

- Adds .editorconfig to make things consistent
- Adds .phpcs.dist.xml and .php-cs-fixers.dist.php for PHP linting
- Adds jsconfig.json to allow IDE eslint to correctly associate aliases

- Updates Composer and NPM commands to include linting
Copy link

cypress bot commented Mar 27, 2024

1 flaky test on run #7827 ↗︎

0 373 46 0 Flakiness 1

Details:

Merge c4c9b93 into 72e3d86...
Project: Bluehost Brand Plugin Commit: 00a9569ee8 ℹ️
Status: Passed Duration: 22:45 💡
Started: Mar 27, 2024 8:39 PM Ended: Mar 27, 2024 9:01 PM
Flakiness  vendor/newfold-labs/wp-module-onboarding/tests/cypress/integration/2-general-onboarding-flow/sitetype-primary.cy.js • 1 flaky test

View Output Video

Test Artifacts
Get Started Site Type Primary > Check for Event API call being made when different categories are selected Test Replay Screenshots Video

Review all test suite changes for PR #1009 ↗︎

Copy link

cypress bot commented Mar 27, 2024

1 flaky test on run #7808 ↗︎

0 373 46 0 Flakiness 1

Details:

Adds code standard checking
Project: Bluehost Brand Plugin Commit: c4c9b9314c
Status: Passed Duration: 22:59 💡
Started: Mar 27, 2024 8:37 PM Ended: Mar 27, 2024 9:00 PM
Flakiness  vendor/newfold-labs/wp-module-ecommerce/tests/cypress/integration/Home/commerceHomePage.cy.js • 1 flaky test

View Output Video

Test Artifacts
Commerce Home Page- Next Steps > Verify Next steps when experience level is novice Test Replay Screenshots Video

Review all test suite changes for PR #1009 ↗︎

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

Successfully merging this pull request may close these issues.

None yet

1 participant