Title of Proposed Theme: BusinessWire

Description: Stylish business theme for Drupal 7, with modernized gui and GPL icons for business applications.

Location of repository: Screenshot and project are located at:
http://drupal.org/sandbox/j0ey/1585582

Drupal Version: 7

CommentFileSizeAuthor
#3 automated review.txt25.15 KBigoen

Comments

pgogy’s picture

Hello,

Ventral (a coding standards checker) reports some issues for you (there are quick to fix, so don't worry about the length)

http://ventral.org/pareview/httpgitdrupalorgsandboxj0ey1585582git

It appears you are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.

README.txt is missing, see the guidelines for in-project documentation.
Remove "version" from the info file, it will be added by drupal.org packaging automatically.

The links above show you what to do to fix these.

Hope this helps

Pat

j0e’s picture

Thanks Pat. Moved to version specific branch, fixed .info and added readme.txt.

Joseph

igoen’s picture

Status: Needs review » Needs work
StatusFileSize
new25.15 KB

Hi, I do automated project review in Ventral.org and your theme have some errors. Average of your errors is your coding style is not standard. I think your theme is clean and elegant. Good work.

( See in ventral.org or in attached file. But better if you go to ventral.org by yourself. Good luck. )

klausi’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

klausi’s picture

Issue summary: View changes

added title and restructured application