Skip to content

Commit

Permalink
Addressing PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
SlayterDev authored and jonathanKingston committed Sep 23, 2021
1 parent d32a339 commit effee12
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 34 deletions.
11 changes: 11 additions & 0 deletions .eslintrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"extends": "standard",
"rules": {
"indent": ["error", 4]
},
"env": {
"commonjs": true,
"browser": true,
"jasmine": true
}
}
17 changes: 0 additions & 17 deletions .eslintrc.js

This file was deleted.

8 changes: 1 addition & 7 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,7 @@ const LISTS_DIR = 'exception-lists'
const defaultConfig = JSON.parse(fs.readFileSync('default-config.json'))
defaultConfig.version = Date.now()

const platforms = [
'extension',
'ios',
'android',
'macos',
'windows'
]
const platforms = require('./platforms')

const nonDefaultLists = []

Expand Down
7 changes: 7 additions & 0 deletions platforms.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
module.exports = [
'extension',
'ios',
'android',
'macos',
'windows'
]
11 changes: 2 additions & 9 deletions tests/config-tests.js
Original file line number Diff line number Diff line change
@@ -1,16 +1,9 @@
const expect = require('chai').expect
const schema = require('./schema.js')
const Ajv = require('ajv').default
const ajv = new Ajv()
const fs = require('fs')

const platforms = [
'extension',
'ios',
'android',
'macos',
'windows'
]
const platforms = require('./../platforms')
const schema = JSON.parse(fs.readFileSync('./tests/schema.json'))

const configs = platforms.map((plat) => {
return JSON.parse(fs.readFileSync(`./generated/${plat}-config.json`))
Expand Down
2 changes: 1 addition & 1 deletion tests/schema.js → tests/schema.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
module.exports = {
{
"root": {
"type": "object",
"properties": {
Expand Down

0 comments on commit effee12

Please sign in to comment.