Do not warn about callback changes until 3.0

2 files changed, 18 insertions, 16 deletions

diff --git a/click/core.py b/click/core.py
index 1c7d3d9..820cc34 100644
--- a/click/core.py
+++ b/click/core.py
@@ -38,11 +38,12 @@ def invoke_param_callback(callback, ctx, param, value):
     args = getattr(code, 'co_argcount', 3)
 
     if args < 3:
-        from warnings import warn
-        warn(Warning('Invoked legacy parameter callback "%s".  The new '
-                     'signature for such callbacks starting with '
-                     'Click 2.0 is (ctx, param, value).'
-                     % callback), stacklevel=3)
+        # This will become a warning in Click 3.0
+        ##from warnings import warn
+        ##warn(Warning('Invoked legacy parameter callback "%s".  The new '
+        ##             'signature for such callbacks starting with '
+        ##             'Click 2.0 is (ctx, param, value).'
+        ##             % callback), stacklevel=3)
         return callback(ctx, value)
     return callback(ctx, param, value)
 
diff --git a/tests/test_compat.py b/tests/test_compat.py
index b2f9899..a46530c 100644
--- a/tests/test_compat.py
+++ b/tests/test_compat.py
@@ -1,16 +1,17 @@
 import click
 
 
-def test_legacy_callbacks(runner):
-    def legacy_callback(ctx, value):
-        return value.upper()
+if click.__version__ >= '3.0':
+    def test_legacy_callbacks(runner):
+        def legacy_callback(ctx, value):
+            return value.upper()
 
-    @click.command()
-    @click.option('--foo', callback=legacy_callback)
-    def cli(foo):
-        click.echo(foo)
+        @click.command()
+        @click.option('--foo', callback=legacy_callback)
+        def cli(foo):
+            click.echo(foo)
 
-    result = runner.invoke(cli, ['--foo', 'wat'])
-    assert result.exit_code == 0
-    assert 'WAT' in result.output
-    assert 'Invoked legacy parameter callback' in result.output
+        result = runner.invoke(cli, ['--foo', 'wat'])
+        assert result.exit_code == 0
+        assert 'WAT' in result.output
+        assert 'Invoked legacy parameter callback' in result.output