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

Port module setup from latest project #32

Merged
merged 8 commits into from
Aug 12, 2023
Merged

Conversation

XedinUnknown
Copy link
Member

Used https://github.com/XedinUnknown/red-acre-test-a-1 as source.

  • No more inc: source files go into src (with some exceptions)
  • The app is now the main module, and all other modules are optional.
  • Modules can simply be autoloaded: no need for explicit factory files.
  • Bootstrap improved, making it more versatile, and usable in tests.
  • Core WP-related things come from WP context, avoiding needless coupling.
  • Tools now work on modules too, adding them to QA.
  • Test base classes for modular layers: test modules alone or together with ease.
  • Tests improved and added.
  • Add Node and WP-CLI to relevant virtual services.
  • Add WordPress directory mapping for easier debugging and more IDE features (PhpStorm WordPress Integration #25).
  • Fix missing get_plugin_data() and other WP plugin APIs.

Used https://github.com/XedinUnknown/red-acre-test-a-1 as source.

- No more `inc`: source files go into `src` (with some exceptions)
- The app is now the main module, and all other modules are optional.
- Modules can simply be autoloaded: no need for explicit factory files.
- Bootstrap improved, making it more versatile, and usable in tests.
- Core WP-related things come from WP context, avoiding needless coupling.
- Tools now work on modules too, adding them to QA.
- Test base classes for modular layers: test modules alone or together with ease.
- Tests improved and added.
- Add Node and WP-CLI to relevant virtual services.
- Add WordPress directory mapping for easier debugging and more IDE features (#25).
- Fix missing `get_plugin_data()` and other WP plugin APIs.
- Deps are included: they may very well contain strings that will be displayed, and need i18n.
- Some other files, like the possibly mapped WP core directory, are excluded, making tasks much faster.
POT files are artefacts.
PO files are generated by translators.
Both are handled during build, and need not be part of the repo.
@XedinUnknown XedinUnknown merged commit 4d8915e into 0.3.x Aug 12, 2023
13 checks passed
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