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

Feature/Refactor #125

Merged
merged 14 commits into from
Jun 27, 2016
Merged

Feature/Refactor #125

merged 14 commits into from
Jun 27, 2016

Conversation

jaswrks
Copy link
Contributor

@jaswrks jaswrks commented Jun 26, 2016

No description provided.

…domain`

- **IMPORTANT:** New required `.build.props` property for lite variation: `project_lite_text_domain`
- If you are missing either of these new properties Phing will fail validation and you will be warned that a property is missing.
- **IMPORTANT:** As noted below, many of the pattern sets were updated in this release. Please review your next build carefully in case there are any new bugs. In particular, please be sure to review the files that make it into the final ZIP distro, as well as the final Lite repo at GitHub. I don't believe there are any problems, but given the number of changes to the pattern sets in this release, it would be worth your while to do a quick review just to be sure.

- **New Wiki Article:** https://github.com/websharks/phings/wiki/Build-Flags-&-Props
- Reorganized all pattern sets. Now using a layered approach that is based on our `.gitignore` and `.gitattributes` file for better consistency and easier maintenance.
- Adding `defaultExcludes="false"` to all `<fileSet>` tags now that there is a more comprehensive approach being taken with exclusions (i.e., pattern sets were drastically improved in this release).
- Changing `-X` argument on `zip` command to the long name for improved clarity; i.e., `--no-extra`.
- Refactor: `_is_wp_core_framework_project` now `_is_core_project`.
- Refactor: `_is_lite_build` now `_doing_lite_build`.
- Refactor: `_has_lite_build_props` now `_has_lite_variation`.
- Adding GitHub credential validator. Now required to finish a release.
- Adding Slack credential validator. Now required to finish a release or update bleeding edge.
- Correcting indentation in all files; i.e., no need to indent inside `<?xml>` tag.
@jaswrks jaswrks mentioned this pull request Jun 27, 2016
10 tasks
@jaswrks jaswrks merged commit d7e06c1 into 000000-dev Jun 27, 2016
@jaswrks jaswrks deleted the feature/refactor branch June 27, 2016 10:43
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

1 participant