Remove gjslint.py

.travis.yml

@@ -1,5 +1,4 @@
-sudo: required
+sudo: false
-dist: trusty
 
 language: node_js
 
@@ -14,7 +13,6 @@ env:
 
 before_install:
   - "npm prune"
-  - "sudo pip install -r requirements.txt"
 
 before_script:
   - "rm src/ol/renderer/webgl/*shader.js"

DEVELOPING.md

@@ -26,10 +26,10 @@ The minimum requirements are:
 * GNU Make
 * Git
 * [Node.js](http://nodejs.org/) (0.10.x or higher)
-* Python 2.6 or 2.7 with a couple of extra modules (see below)
+* Python 2.6 or 2.7
 * Java 7 (JRE and JDK)
 
-The executables `git`, `node`, `python` and `java` should be in your `PATH`.
+The executables `git`, `node`, and `java` should be in your `PATH`.
 
 You can check your configuration by running:
 
@@ -39,17 +39,6 @@ To install the Node.js dependencies run
 
     $ npm install
 
-To install the extra Python modules, run:
-
-    $ sudo pip install -r requirements.txt
-or
-
-    $ cat requirements.txt | sudo xargs easy_install
-
-depending on your OS and Python installation.
-
-(You can also install the Python modules in a Python virtual environment if you want to.)
-
 ## Working with the build tool
 
 As an ol3 developer you will use `make` to run build targets defined in the

Makefile

@@ -93,7 +93,6 @@ compile-examples: build/compiled-examples/all.combined.js
 
 .PHONY: clean
 clean:
-	rm -f build/timestamps/gjslint-timestamp
 	rm -f build/timestamps/eslint-timestamp
 	rm -f build/timestamps/check-*-timestamp
 	rm -f build/ol.css
@@ -120,8 +119,7 @@ examples: $(BUILD_EXAMPLES)
 install: build/timestamps/node-modules-timestamp
 
 .PHONY: lint
-lint: build/timestamps/gjslint-timestamp \
+lint: build/timestamps/eslint-timestamp \
-      build/timestamps/eslint-timestamp \
       build/timestamps/check-requires-timestamp \
       build/timestamps/check-whitespace-timestamp
 
@@ -232,13 +230,6 @@ build/timestamps/jsdoc-$(BRANCH)-timestamp: config/jsdoc/api/index.md \
 	./node_modules/.bin/jsdoc config/jsdoc/api/index.md -c config/jsdoc/api/conf.json -d $(BUILD_HOSTED)/apidoc
 	@touch $@
 
-build/timestamps/gjslint-timestamp: $(SRC_JS) $(SPEC_JS) $(SPEC_RENDERING_JS) \
-                                    $(EXAMPLES_JS)
-	@mkdir -p $(@D)
-	@echo "Running gjslint..."
-	@gjslint --jslint_error=all --disable 0110 --custom_jsdoc_tags=event,fires,function,classdesc,api,observable --strict $?
-	@touch $@
-
 $(BUILD_HOSTED_EXAMPLES_JS): $(BUILD_HOSTED)/examples/%.js: build/examples/%.js
 	@mkdir -p $(@D)
 	@python bin/split-example.py $< $(@D)

requirements.txt

@@ -1 +0,0 @@
-http://closure-linter.googlecode.com/files/closure_linter-latest.tar.gz

test/spec/ol/geom/linestring.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.LineString', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var lineString = new ol.geom.LineString(null);
+      return new ol.geom.LineString(null);
-      lineString = lineString; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/geom/multilinestring.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.MultiLineString', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var multiLineString = new ol.geom.MultiLineString(null);
+      return new ol.geom.MultiLineString(null);
-      multiLineString = multiLineString; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/geom/multipoint.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.MultiPoint', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var multiPoint = new ol.geom.MultiPoint(null);
+      return new ol.geom.MultiPoint(null);
-      multiPoint = multiPoint; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/geom/multipolygon.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.MultiPolygon', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var multiPolygon = new ol.geom.MultiPolygon(null);
+      return new ol.geom.MultiPolygon(null);
-      multiPolygon = multiPolygon; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/geom/point.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.Point', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var point = new ol.geom.Point(null);
+      return new ol.geom.Point(null);
-      point = point; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/geom/polygon.test.js

@@ -5,8 +5,7 @@ describe('ol.geom.Polygon', function() {
 
   it('can be constructed with a null geometry', function() {
     expect(function() {
-      var polygon = new ol.geom.Polygon(null);
+      return new ol.geom.Polygon(null);
-      polygon = polygon; // suppress gjslint warning
     }).not.to.throwException();
   });
 

test/spec/ol/tilerange.test.js

@@ -59,9 +59,7 @@ describe('ol.TileRange', function() {
 
     describe('with mixed z', function() {
       expect(function() {
-        var tileRange = new ol.TileRange.boundingTileRange(
+        return new ol.TileRange.boundingTileRange([3, 1, 3], [4, 2, 0]);
-            [3, 1, 3], [4, 2, 0]);
-        tileRange = tileRange; // suppress gjslint warning about unused variable
       }).to.throwException();
     });
   });