Hello,

I created a theme "Tej", and I'm pleased to apply for full project permission for it.

Tej is a new subtheme of the responsive Omega theme.
Tej integrates the Color module and comes with 6 color schemes.
RTL is supported in Tej theme.

Project page: http://drupal.org/sandbox/ahwebd/1372650
Git: git clone http://git.drupal.org/sandbox/ahwebd/1372650.git tej

Tej is now developed for Drupal 7.x and for 3.x branch of Omega.

Regards

CommentFileSizeAuthor
#9 drupalcs-result.txt645 bytesklausi
tej.png65.01 KBahwebd

Comments

jinlong’s picture

jinlong commented
Status: Needs review » Needs work

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.
Review of the master branch:

  • Drupal Code Sniffer has found some code style issues (please check the Drupal coding standards):
    
FILE: ...pareview/sites/all/modules/pareview_temp/test_candidate/color/color.inc
--------------------------------------------------------------------------------
FOUND 203 ERROR(S) AFFECTING 203 LINE(S)
--------------------------------------------------------------------------------
   2 | ERROR | You must use "/**" style comments for a file comment
  47 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  48 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  49 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  50 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  51 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  52 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  53 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  54 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  55 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  56 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  57 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  58 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  59 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  60 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  61 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  62 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  63 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  64 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  65 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  66 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  67 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  68 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  69 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  70 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  71 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  72 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  73 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  74 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  75 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  76 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  77 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  78 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  84 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  85 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  86 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  87 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  88 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  89 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  90 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  91 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  92 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  93 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  94 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  95 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  96 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  97 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  98 | ERROR | Spaces must be used to indent lines; tabs are not allowed
  99 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 100 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 101 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 102 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 103 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 104 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 105 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 106 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 107 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 108 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 109 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 110 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 111 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 112 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 113 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 114 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 115 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 116 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 117 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 118 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 121 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 122 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 123 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 124 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 125 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 126 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 127 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 128 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 129 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 130 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 131 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 132 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 133 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 134 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 135 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 136 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 137 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 138 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 139 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 140 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 141 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 142 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 143 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 144 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 145 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 146 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 147 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 148 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 149 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 150 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 151 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 152 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 153 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 154 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 158 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 159 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 160 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 161 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 162 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 163 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 164 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 165 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 166 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 167 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 168 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 169 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 170 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 171 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 172 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 173 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 174 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 175 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 176 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 177 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 178 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 179 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 180 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 181 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 182 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 183 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 184 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 185 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 186 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 187 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 188 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 189 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 190 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 191 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 195 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 196 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 197 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 198 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 199 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 200 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 201 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 202 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 203 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 204 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 205 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 206 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 207 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 208 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 209 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 210 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 211 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 212 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 213 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 214 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 215 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 216 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 217 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 218 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 219 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 220 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 221 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 222 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 223 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 224 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 225 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 226 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 227 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 228 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 232 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 233 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 234 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 235 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 236 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 237 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 238 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 239 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 240 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 241 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 242 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 243 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 244 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 245 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 246 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 247 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 248 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 249 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 250 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 251 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 252 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 253 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 254 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 255 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 256 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 257 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 258 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 259 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 260 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 261 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 262 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 263 | ERROR | Spaces must be used to indent lines; tabs are not allowed
 307 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...-pareview/sites/all/modules/pareview_temp/test_candidate/css/README.txt
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 11 WARNING(S) AFFECTING 12 LINE(S)
--------------------------------------------------------------------------------
  1 | WARNING | Line exceeds 80 characters; contains 90 characters
  2 | WARNING | Line exceeds 80 characters; contains 81 characters
  3 | WARNING | Line exceeds 80 characters; contains 88 characters
  4 | WARNING | Line exceeds 80 characters; contains 89 characters
  5 | WARNING | Line exceeds 80 characters; contains 89 characters
  6 | WARNING | Line exceeds 80 characters; contains 89 characters
  8 | WARNING | Line exceeds 80 characters; contains 90 characters
 10 | WARNING | Line exceeds 80 characters; contains 90 characters
 12 | WARNING | Line exceeds 80 characters; contains 90 characters
 19 | WARNING | Line exceeds 80 characters; contains 90 characters
 21 | WARNING | Line exceeds 80 characters; contains 90 characters
 23 | ERROR   | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...-pareview/sites/all/modules/pareview_temp/test_candidate/css/colors.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 125 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...-pareview/sites/all/modules/pareview_temp/test_candidate/css/global.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 292 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...l/modules/pareview_temp/test_candidate/css/tej-alpha-default-narrow.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 1 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...l/modules/pareview_temp/test_candidate/css/tej-alpha-default-normal.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 1 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...all/modules/pareview_temp/test_candidate/css/tej-alpha-default-wide.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 1 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...ites/all/modules/pareview_temp/test_candidate/css/tej-alpha-default.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 1 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...ew/sites/all/modules/pareview_temp/test_candidate/preprocess/README.txt
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 14 WARNING(S) AFFECTING 15 LINE(S)
--------------------------------------------------------------------------------
  1 | WARNING | Line exceeds 80 characters; contains 90 characters
  2 | WARNING | Line exceeds 80 characters; contains 81 characters
  3 | WARNING | Line exceeds 80 characters; contains 88 characters
  4 | WARNING | Line exceeds 80 characters; contains 89 characters
  5 | WARNING | Line exceeds 80 characters; contains 89 characters
  6 | WARNING | Line exceeds 80 characters; contains 89 characters
  8 | WARNING | Line exceeds 80 characters; contains 90 characters
 10 | WARNING | Line exceeds 80 characters; contains 90 characters
 12 | WARNING | Line exceeds 80 characters; contains 90 characters
 19 | WARNING | Line exceeds 80 characters; contains 90 characters
 21 | WARNING | Line exceeds 80 characters; contains 90 characters
 23 | WARNING | Line exceeds 80 characters; contains 88 characters
 33 | WARNING | Line exceeds 80 characters; contains 89 characters
 34 | WARNING | Line exceeds 80 characters; contains 90 characters
 39 | ERROR   | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...eview/sites/all/modules/pareview_temp/test_candidate/process/README.txt
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 14 WARNING(S) AFFECTING 15 LINE(S)
--------------------------------------------------------------------------------
  1 | WARNING | Line exceeds 80 characters; contains 90 characters
  2 | WARNING | Line exceeds 80 characters; contains 81 characters
  3 | WARNING | Line exceeds 80 characters; contains 88 characters
  4 | WARNING | Line exceeds 80 characters; contains 89 characters
  5 | WARNING | Line exceeds 80 characters; contains 89 characters
  6 | WARNING | Line exceeds 80 characters; contains 89 characters
  8 | WARNING | Line exceeds 80 characters; contains 90 characters
 10 | WARNING | Line exceeds 80 characters; contains 90 characters
 12 | WARNING | Line exceeds 80 characters; contains 90 characters
 19 | WARNING | Line exceeds 80 characters; contains 90 characters
 21 | WARNING | Line exceeds 80 characters; contains 90 characters
 23 | WARNING | Line exceeds 80 characters; contains 85 characters
 33 | WARNING | Line exceeds 80 characters; contains 86 characters
 34 | WARNING | Line exceeds 80 characters; contains 87 characters
 39 | ERROR   | Files must end in a single new line character
--------------------------------------------------------------------------------


FILE: ...-7-pareview/sites/all/modules/pareview_temp/test_candidate/template.php
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AND 4 WARNING(S) AFFECTING 6 LINE(S)
--------------------------------------------------------------------------------
  5 | WARNING | Line exceeds 80 characters; contains 86 characters
  6 | WARNING | Line exceeds 80 characters; contains 93 characters
  8 | ERROR   | Whitespace found at end of line
  9 | WARNING | Line exceeds 80 characters; contains 93 characters
 10 | WARNING | Line exceeds 80 characters; contains 97 characters
 30 | ERROR   | You must use "/**" style comments for a function comment
--------------------------------------------------------------------------------


FILE: ...iew/sites/all/modules/pareview_temp/test_candidate/templates/README.txt
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 13 WARNING(S) AFFECTING 14 LINE(S)
--------------------------------------------------------------------------------
  1 | WARNING | Line exceeds 80 characters; contains 90 characters
  2 | WARNING | Line exceeds 80 characters; contains 81 characters
  3 | WARNING | Line exceeds 80 characters; contains 88 characters
  4 | WARNING | Line exceeds 80 characters; contains 89 characters
  5 | WARNING | Line exceeds 80 characters; contains 89 characters
  6 | WARNING | Line exceeds 80 characters; contains 89 characters
  8 | WARNING | Line exceeds 80 characters; contains 90 characters
 10 | WARNING | Line exceeds 80 characters; contains 90 characters
 12 | WARNING | Line exceeds 80 characters; contains 90 characters
 19 | WARNING | Line exceeds 80 characters; contains 90 characters
 21 | WARNING | Line exceeds 80 characters; contains 89 characters
 22 | WARNING | Line exceeds 80 characters; contains 88 characters
 25 | WARNING | Line exceeds 80 characters; contains 87 characters
 26 | ERROR   | Files must end in a single new line character
--------------------------------------------------------------------------------
  • 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.
  • Bad line endings were found, always use unix style terminators. See http://drupal.org/coding-standards#indenting
    ./css/tej-alpha-default-narrow.css: ASCII C program text, with no line terminators
./css/tej-alpha-default-wide.css:   ASCII C program text, with no line terminators
./css/tej-alpha-default.css:        ASCII C program text, with no line terminators
./css/tej-alpha-default-normal.css: ASCII C program text, with no line terminators
./color/preview.html:               ASCII text, with no line terminators

This automated report was generated with PAReview.sh, your friendly project application review script. Go and review some other project applications, so we can get back to yours sooner.

Source: http://ventral.org/pareview - PAReview.sh online service

ahwebd’s picture

ahwebd commented

Thanks for the quick reply,

I was just in the cleaning process after I found http://ventral.org/pareview

When I finish I'll make a new commit.

ahwebd’s picture

ahwebd commented
Status: Needs work » Needs review

Can you please check now

drupalnetworks’s picture

drupalnetworks commented
  • Remove "version" from the info file, it will be added by drupal.org packaging automatically.
ahwebd’s picture

ahwebd commented

version removed

patrickd’s picture

patrickd commented
Status: Needs review » Needs work

Please give more detailed information on your project page (some screenshots also would be nice)
Same with your readme, one line is not pretty informative ;-) what about installation? configuration? any Theme-specific thinks a themer should know?

I'm not an expert about themes but this looks pretty good to me, so I'll RTBC this after you made created a nice project page and readme!

ahwebd’s picture

ahwebd commented
Status: Needs work » Needs review

Project page and README.txt updated,

patrickd’s picture

patrickd commented
Status: Needs review » Reviewed & tested by the community

I think this is ready to go.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new drupalcs-result.txt645 bytes

Review of the 7.x-1.x branch:

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. Go and review some other project applications, so we can get back to yours sooner.

manual review:
There is not really much to review here, so I think your project is too short to approve you as git vetted user. We are currently discussing how much code we need, but everything with less than 5 functions or 120 lines of code cannot be seriously reviewed. So I have promoted this single project for you and you can get back to us when you have a new project that you want to promote.

Now that this experimental project has been promoted, you'll need to update the URL of your remote repository or reclone it.

Thanks for your contribution, ahwebd! Welcome to the community of project contributors on drupal.org.

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

As you continue to work on your module, keep in mind: Commit messages - providing history and credit and Release naming conventions.

ahwebd’s picture

ahwebd commented

Thank you!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.