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.

Sign up for GitHub

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

Jump to bottom

Remove grunt from plugin scaffolding #337

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Remove grunt from plugin scaffolding #337

wants to merge 3 commits into from

Conversation

ernilambar
Copy link
Member

@ernilambar ernilambar commented May 9, 2024 •
edited
Loading

Fixes #140

  • plugin-gruntfile.mustache is removed.
  • plugin-packages.mustache is removed.
  • plugin-composer.mustache is added.
  • vendor/ is added in plugin-gitignore.mustache
  • In composer wp-cli/i18n-command package is added. Script makepot is kept to generate POT file so that user can generate POT file like this command composer run makepot

@ernilambar ernilambar marked this pull request as ready for review May 9, 2024 05:52
@ernilambar ernilambar requested a review from a team as a code owner May 9, 2024 05:52
@ernilambar ernilambar mentioned this pull request May 9, 2024
Open
@ernilambar @swissspidy
Remove vendor bin path
a0fd65d 
Co-authored-by: Pascal Birchler <pascal.birchler@gmail.com>
@ernilambar ernilambar mentioned this pull request Jun 10, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@swissspidy swissspidy swissspidy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

(Partly) remove Grunt tasks
2 participants
@ernilambar @swissspidy