Fix/build deploy

[cadic]

Description of the Change

1. Added .distignore
2. Added composer install —no-dev command to the build step
3. Updated textdomain

Closes #6

Verification Process

The release process was verified by replicating commands which the action will perform:

1. npm install
2. npm run build
3. composer install --no-dev
4. rsync -rc --exclude-from=".distignore" . trunk/ --delete --delete-excluded

The resulting trunk/ folder contains correct list of files:

Checklist:

• I have read the CONTRIBUTING document.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests passed.

Changelog Entry

Fix - Deploy to WordPress.org action

Credits

Props @cadic

[jeffpaul]

can probably remove the gitattributes file

[jeffpaul]

what's this here for?

[cadic]

Added this in case plugin dirtro zip which was made by npm run archive script is present. It will never appear within the current push-deploy workflow, should we remove it?

[cadic]

Will delete it for now, we can add it later when it's really required

[jeffpaul]

Yeah I don't think I've seen that creep in, so probably fine to remove but good idea to have ready in case that does end up happening here

[jeffpaul]

do we need to add this same step to the readme/asset action as well?
https://github.com/10up/sophi-debug-bar/blob/5f13d331ff27a21802d62188d6561f93ad44b14c/.github/workflows/wordpress-plugin-asset-update.yml#L12-L15

[cadic]

Correct, we also need vendor folder to exist (otherwise the deploy script might consider "Other files have been modified")

[jeffpaul]

Some comments/updates needed, thanks for getting this started so quickly!

[jeffpaul]

awesome, thank you!