Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Aug 2015 at 19:52 UTC
Updated:
8 Sep 2015 at 08:55 UTC
Jump to comment: Most recent
Comments
Comment #2
PA robot commentedWe are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #3
sheldonkreger commentedManual Review
Individual user account
YES
No duplication
UNSURE - Please explain to me why this can't be built atop the main Deploy module project.
Master Branch
NO - needs release candidate branch (see https://www.drupal.org/node/1127732 step 2)
Licensing
YES
3rd party assets/code
YES
README.txt/README.md
YES
Code long/complex enough for review
YES
Secure code
YES
Coding style & Drupal API usage
YES - no issues found http://pareview.sh/pareview/httpgitdrupalorgsandboxlucaskaneddeployplusgit
Comment #4
artematem commentedsheldonkreger,
As mentioned in this #2224901 Add hook_deploy_auto_plan_entity_save_alter comment, they want to keep Deploy module simple and straight forward. And patches that adds features from this module to Deploy are also created in:
#2499783 Add plan clean up functionality.
#2512740 Detail plan items view
#1604938 Add hook_action_info() integration for use with Views Bulk Operations
So I made it as separate module, as for some purposes simple and straight forward Deploy module is enough.
Not sure about
I have branch 7.x-1.x. And release candidate can be created after promoting module to full project, right?
Comment #5
artematem commentedComment #6
sheldonkreger commentedI think it makes sense to list this as a separate project. Marking as RTBC.
Comment #7
artematem commentedPromoted to full project.