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

Closed
wants to merge 4 commits into from
Closed

Adds code standard checking #1009

wants to merge 4 commits 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

Bluehost Brand Plugin    Run #10918

Run Properties:  status check failed Failed #10918  •  git commit 90e20d67e6 ℹ️: Merge a2dc9eafe0b973b00170052b6a9e2f3b36d6cb51 into f94f7e89f4c720149255a8f25847...
Project Bluehost Brand Plugin
Branch Review feature/code-standards-2
Run status status check failed Failed #10918
Run duration 36m 45s
Commit git commit 90e20d67e6 ℹ️: Merge a2dc9eafe0b973b00170052b6a9e2f3b36d6cb51 into f94f7e89f4c720149255a8f25847...
Committer Brad Parbs
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 36
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 416
View all changes introduced in this branch ↗︎

Tests for review

Failed  vendor/newfold-labs/wp-module-migration/tests/cypress/integration/Migration/landIntoMigrationFromAM.cy.js • 1 failed test

View Output Video

Test Artifacts
Verify Migration- emulating AM flow > Verify Migration page is loaded Test Replay Screenshots Video

Copy link

cypress bot commented Mar 27, 2024

Bluehost Brand Plugin    Run #10903

Run Properties:  status check failed Failed #10903  •  git commit a2dc9eafe0: Actually run both commands
Project Bluehost Brand Plugin
Branch Review feature/code-standards-2
Run status status check failed Failed #10903
Run duration 31m 18s
Commit git commit a2dc9eafe0: Actually run both commands
Committer Brad Parbs
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 36
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 416
View all changes introduced in this branch ↗︎

Tests for review

Failed  vendor/newfold-labs/wp-module-migration/tests/cypress/integration/Migration/landIntoMigrationFromAM.cy.js • 1 failed test

View Output Video

Test Artifacts
Verify Migration- emulating AM flow > Verify Migration page is loaded Test Replay Screenshots Video

@circlecube circlecube marked this pull request as draft June 19, 2024 18:40
@bradp bradp marked this pull request as ready for review October 2, 2024 00:52
@bradp
Copy link
Contributor Author

bradp commented Oct 2, 2024

Checks failing are to be expected until #852 is merged into this branch.

@circlecube circlecube added this to the October 9, 2024 Release milestone Oct 3, 2024
@wpscholar wpscholar removed this from the October 9, 2024 Release milestone Oct 16, 2024
@circlecube circlecube marked this pull request as draft November 8, 2024 22:06
@circlecube
Copy link
Member

Based on the lack of updates on this and the now number of conflicts, I think we can close this and reopen or restart if we have the need in the future.

@circlecube circlecube closed this Nov 12, 2024
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.

3 participants