Skip to content
Permalink
Browse files

Allow keyword arguments for all arguments in PyQGIS bindings

Previously this was only enabled for optional arguments (i.e.
those with default values). Enabling them for all arguments
allows for more readable PyQGIS code, and there seems no
downside given that we already have this support partly enabled.

The consequence of this change is that when 3.0 API is frozen
the freeze must also include the naming of function arguments,
since that's effectively now part of public API.
  • Loading branch information
nyalldawson committed Jun 25, 2017
1 parent 69af2f5 commit 2b7b0db7476cd14a134e9cc91ef7f0ebc1fd7a6f
Showing with 4 additions and 4 deletions.
  1. +1 −1 python/analysis/analysis.sip
  2. +1 −1 python/core/core.sip
  3. +1 −1 python/gui/gui.sip
  4. +1 −1 python/server/server.sip
@@ -1,5 +1,5 @@
%Module(name=qgis._analysis,
keyword_arguments="Optional")
keyword_arguments="All")

%Import QtCore/QtCoremod.sip
%Import QtGui/QtGuimod.sip
@@ -1,5 +1,5 @@
%Module(name=qgis._core,
keyword_arguments="Optional")
keyword_arguments="All")

%ModuleCode

@@ -1,5 +1,5 @@
%Module(name=qgis._gui,
keyword_arguments="Optional")
keyword_arguments="All")

%Feature HAVE_QSCI_SIP

@@ -1,5 +1,5 @@
%Module(name=qgis._server,
keyword_arguments="Optional")
keyword_arguments="All")


%Import QtCore/QtCoremod.sip

0 comments on commit 2b7b0db

Please sign in to comment.
You can’t perform that action at this time.