Use beta tag

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ol",
-  "version": "6.0.0-beta.11",
+  "version": "6.0.0-beta.10",
   "description": "OpenLayers mapping library",
   "keywords": [
     "map",

rendering/test.js

@@ -357,7 +357,7 @@ if (require.main === module) {
     option('puppeteer-args', {
       describe: 'Additional args for Puppeteer',
       type: 'array',
-      default: process.env.CI ? ['--no-sandbox', '--disable-setuid-sandbox', '--disable-dev-shm-usage'] : []
+      default: process.env.CI ? ['--no-sandbox', '--disable-setuid-sandbox'] : []
     }).
     parse();
 

src/ol/format/MVT.js

@@ -31,7 +31,6 @@ import {get} from '../proj.js';
  * @property {string} [geometryName='geometry'] Geometry name to use when creating features.
  * @property {string} [layerName='layer'] Name of the feature attribute that holds the layer name.
  * @property {Array<string>} [layers] Layers to read features from. If not provided, features will be read from all
- * @property {string} [idProperty] Optional property that will be assigned as the feature id and removed from the properties.
  * layers.
  */
 
@@ -85,12 +84,6 @@ class MVT extends FeatureFormat {
      */
     this.layers_ = options.layers ? options.layers : null;
 
-    /**
-     * @private
-     * @type {string}
-     */
-    this.idProperty_ = options.idProperty;
-
   }
 
   /**
@@ -171,16 +164,8 @@ class MVT extends FeatureFormat {
     }
 
     let feature;
+    const id = rawFeature.id;
     const values = rawFeature.properties;
-
-    let id;
-    if (!this.idProperty_) {
-      id = rawFeature.id;
-    } else {
-      id = values[this.idProperty_];
-      delete values[this.idProperty_];
-    }
-
     values[this.layerName_] = rawFeature.layer.name;
 
     const flatCoordinates = [];

tasks/publish.sh

@@ -76,7 +76,7 @@ main() {
   npm install
   npm run build-package
   cd ${BUILT_PACKAGE}
-  npm publish
+  npm publish --tag beta
 }
 
 if test ${#} -ne 1; then

test/spec/ol/format/mvt.test.js

@@ -78,44 +78,6 @@ where('ArrayBuffer.isView').describe('ol.format.MVT', function() {
       expect(features[0].getId()).to.be(2);
     });
 
-    it('accepts custom idProperty', function() {
-      const format = new MVT({
-        featureClass: Feature,
-        layers: ['poi_label'],
-        idProperty: 'osm_id'
-      });
-      const features = format.readFeatures(data, options);
-
-      const first = features[0];
-      expect(first.getId()).to.be(1000000057590683);
-      expect(first.get('osm_id')).to.be(undefined);
-    });
-
-    it('accepts custom idProperty (render features)', function() {
-      const format = new MVT({
-        layers: ['poi_label'],
-        idProperty: 'osm_id'
-      });
-
-      const features = format.readFeatures(data, options);
-
-      const first = features[0];
-      expect(first.getId()).to.be(1000000057590683);
-      expect(first.get('osm_id')).to.be(undefined);
-    });
-
-    it('works if you provide a bogus idProperty', function() {
-      const format = new MVT({
-        layers: ['poi_label'],
-        idProperty: 'bogus'
-      });
-
-      const features = format.readFeatures(data, options);
-
-      const first = features[0];
-      expect(first.getId()).to.be(undefined);
-    });
-
   });
 
 });