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 eslint-plugin-wpvip to 0.5.8, WP Prettier, GitHub Action improvements #323

Merged
merged 8 commits into from Jul 26, 2023

Conversation

chriszarate
Copy link
Member

Description

  • Update eslint-plugin-wpvip to 0.5.8 which contains additional lint rules and expands Prettier config to work with JSON, Markdown, and YAML files.
  • Install wp-prettier
  • Remove Babel, since there are no features in the code that require it. We can separately introduce TypeScript
  • Add .editorconfig
  • Standardize GitHub Action workflows to reduce unnecessary runs and run Prettier check.
  • Standardize npm scripts
  • Add new NPM publishing workflows
  • Update code to fix lint errors
  • Apply Prettier fixes

Steps to Test

npm test

Copy link

@abdullah-kasim abdullah-kasim left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@chriszarate chriszarate merged commit c7fee29 into trunk Jul 26, 2023
2 checks passed
@chriszarate chriszarate deleted the update/eslint-wpvip-0.5.8 branch July 26, 2023 14:39
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.

None yet

2 participants