Re: [Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
-+-
 Reporter:  sebastian.pawlus@…   |Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |  Version:
Component:  Testing framework|  1.7-alpha-1
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Claude Paroz ):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"4622d23ccf50a72432ff0c7416a4d6e0789c4f0b"]:
 {{{
 #!CommitTicketReference repository=""
 revision="4622d23ccf50a72432ff0c7416a4d6e0789c4f0b"
 Fixed #22056 -- Omitted non-package directories when collecting tests

 This allows us to ignore empty dirs or other unrelated dirs.
 Thanks Sebastian Pawluś for the report and the initial patch.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/084.a7f81d5bbadd08740213ce53872e4903%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
-+-
 Reporter:  sebastian.pawlus@…   |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:
Component:  Testing framework|  1.7-alpha-1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by claudep):

 * has_patch:  0 => 1
 * stage:  Unreviewed => Accepted


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/084.7d4b7f86207a8198f45de403138ef0bd%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
-+-
 Reporter:  sebastian.pawlus@…   |Owner:  nobody
 Type:   |   Status:  new
  Cleanup/optimization   |  Version:
Component:  Testing framework|  1.7-alpha-1
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
Has patch:  0|  Unreviewed
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by AeroNotix):

 * type:  Uncategorized => Cleanup/optimization


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/084.3659bf14e4ec35444815e123797f7f30%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
-+-
 Reporter:  sebastian.pawlus@…   |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Testing framework|  Version:
 Severity:  Normal   |  1.7-alpha-1
 Keywords:   |   Resolution:
Has patch:  0| Triage Stage:
  Needs tests:  0|  Unreviewed
Easy pickings:  0|  Needs documentation:  0
 |  Patch needs improvement:  0
 |UI/UX:  0
-+-

Comment (by timo):

 This may be find. Personally, I use a post-checkout hook:
 {{{
 # Delete .pyc files and empty directories.
 find . -name "*.pyc" -delete
 find . -type d -empty -delete
 }}}
 We could at least document that if there's any drawback to the PR.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/084.2322e61c8d6816b2dc3e77985c5074ae%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
-+-
 Reporter:  sebastian.pawlus@…   |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Testing framework|  Version:
 Severity:  Normal   |  1.7-alpha-1
 Keywords:   |   Resolution:
Has patch:  0| Triage Stage:
  Needs tests:  0|  Unreviewed
Easy pickings:  0|  Needs documentation:  0
 |  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by sebastian.pawlus):

 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 Pull request
 https://github.com/django/django/pull/2292

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/084.f99fe7f40870069a3c8703d604226284%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

[Django] #22056: Empty directories shouldn't be included to test modules

[Django]

#22056: Empty directories shouldn't be included to test modules
+-
 Reporter:  sebastian.pawlus@…  |  Owner:  nobody
 Type:  Uncategorized   | Status:  new
Component:  Testing framework   |Version:  1.7-alpha-1
 Severity:  Normal  |   Keywords:
 Triage Stage:  Unreviewed  |  Has patch:  0
Easy pickings:  0   |  UI/UX:  0
+-
 Currently empty directories are included to the tests modules. It's
 incredibly annoying especially when switching branches and git leaves
 empty directories.

 To reproduce just create an empty directory inside django/tests/

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/069.02ad32f97130db2b4674e746295cc360%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.