Re: [Django] #14098: Add "--skip" option to inspectdb.
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
Type: | Status: new
Uncategorized | Component: Core (Management
Milestone: | commands)
Version: 1.2 | Severity: Normal
Resolution: | Keywords:
Triage Stage: Accepted | Has patch: 1
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 1 | Easy pickings: 0
-------------------------------------+-------------------------------------
Changes (by patchhammer):
* needs_better_patch: 0 => 1
* type: => Uncategorized
* severity: => Normal
* easy: => 0
Comment:
skip.diff fails to apply cleanly on to trunk
--
Ticket URL: <http://code.djangoproject.com/ticket/14098#comment:4>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
Type: | Status: new
Uncategorized | Component: Core (Management
Milestone: | commands)
Version: 1.2 | Severity: Normal
Resolution: | Keywords:
Triage Stage: Accepted | Has patch: 1
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 1 | Easy pickings: 0
-------------------------------------+-------------------------------------
Comment (by adamv):
If there is still interest in accepting this, I can rework the patch;
otherwise a maintainer ought to close it.
--
Ticket URL: <http://code.djangoproject.com/ticket/14098#comment:5>
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
feature | Component: Core (Management
Version: 1.2 | Severity: Normal
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 1 | Easy pickings: 0
-------------------------------------+-------------------------------------
Changes (by ramiro):
* keywords: => inspectdb
* ui_ux: => 0
--
Ticket URL: <https://code.djangoproject.com/ticket/14098#comment:7>
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
Component: Core (Management | Version: master
commands) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: inspectdb | Needs documentation: 0
Has patch: 1 | Patch needs improvement: 0
Needs tests: 1 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by claudep):
* needs_better_patch: 1 => 0
* version: 1.2 => master
* needs_tests: 0 => 1
Comment:
The attached patch, instead of adding a new command parameter, just output
in the result that some table was not introspected. For example:
{{{
# Unable to inspect table v_test
# The error was: View 'test_.v_test' references invalid table(s) or
column(s) or function(s) or definer/invoker of view lack rights to use
them
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/14098#comment:8>
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: inspectdb | Needs documentation: 0
Has patch: 1 | Patch needs improvement: 0
Needs tests: 1 | UI/UX: 0
Easy pickings: 0 |
-------------------------------------+-------------------------------------
* cc: anubhav9042@… (added)
--
Ticket URL: <https://code.djangoproject.com/ticket/14098#comment:9>
Django
-------------------------------------+-------------------------------------
Reporter: adamv | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) |
Keywords: inspectdb | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by claudep):
* needs_tests: 1 => 0
Comment:
[https://github.com/django/django/pull/6193 PR]
--
Ticket URL: <https://code.djangoproject.com/ticket/14098#comment:10>