Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
15 Feb 2014 at 18:10 UTC
Updated:
27 Jun 2014 at 20:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
chakrapani commentedHi lauriemustafic,
The repository is empty. Please follow the instructions in the version control tab and commit code to drupal's git repository rather than providing github's repository url.
Comment #2
lauriemustafic commentedMissed that some how... It's there now.
Comment #3
j.branson commentedHi lauriemustafic, thanks for contributing -- first thing, the git rep you've included in the project description above is to github, you should follow drupal.org instructions and use the drupal.org repository: https://drupal.org/node/1011698. When doing that you need to run git checkout 7.x-1.x and then delete master branch.
Check out the auto review based on Coder and PHP_CodeSniffer:
http://pareview.sh/pareview/httpgitdrupalorgsandboxlauriemustafic2197069git
It's important to note that this review is based on your drupal.org repository, so perhaps you've fixed some things on your github.com repository, but that's why the instructions are important so that we are all pulling form the same repo.
Anyways, the PAReview kicks up a lot of issues, the easiest to address is no README.txt. From there You'll have to go through line by line, A tip from having done this before is to start at the bottom so that the line # references in the PAReview stay relevant.
Good luck!
Comment #4
PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #5
lauriemustafic commented