Skip to content

Commit

Permalink
Fixup: Address automated lint comments
Browse files Browse the repository at this point in the history
  • Loading branch information
ianguerin committed Jun 5, 2024
1 parent b7b3da1 commit 9b36b2d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 6 deletions.
5 changes: 3 additions & 2 deletions src/js/collections/maps/AssetCategories.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@ define([
/**
* Gets a single, flattened MapAssets collection from the AssetCategory
* group in the collection.
* @returns {MapAssets}
* @returns {MapAssets} A single MapAssets collection constructed from
* groups of MapAsset models.
*/
getMapAssetsFlat() {
return new MapAssets(
Expand All @@ -49,7 +50,7 @@ define([

/**
* Gets an array of MapAssets, one from each AssetCategory model.
* @returns {MapAssets[]}
* @returns {MapAssets[]} A list of MapAssets collections.
*/
getMapAssets() {
return this.map((assetCategory) => {

Check failure on line 56 in src/js/collections/maps/AssetCategories.js

View workflow job for this annotation

GitHub Actions / Run All Checks

[eslint] reported by reviewdog 🐶 Unexpected block statement surrounding arrow body; move the returned value immediately after the `=>`. Raw Output: {"ruleId":"arrow-body-style","severity":2,"message":"Unexpected block statement surrounding arrow body; move the returned value immediately after the `=>`.","line":56,"column":44,"nodeType":"ArrowFunctionExpression","messageId":"unexpectedSingleBlock","endLine":58,"endColumn":10,"fix":{"range":[1808,1868],"text":"assetCategory.get(\"mapAssets\")"}}
Expand Down
2 changes: 1 addition & 1 deletion src/js/models/maps/Map.js
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,7 @@ define([
* Reset the visibility of all layers to the value that was in the intial
* configuration.
*/
resetLayerVisibility: function () {
resetLayerVisibility() {
this.get("allLayers").forEach((layer) => {
layer.set("visible", layer.get("originalVisibility"));
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,12 @@ define([
});

it("shows zoom presets UI when enabled in config", () => {
const view = new ViewfinderView({
model: new Map({ zoomPresets: [{}], allLayers: { models: [] } })
}, { parse: true });
const view = new ViewfinderView(
{
model: new Map({ zoomPresets: [{}], allLayers: { models: [] } }),
},
{ parse: true },
);

const harness = new ViewfinderViewHarness(view);
view.render();
Expand Down

0 comments on commit 9b36b2d

Please sign in to comment.