mirror of
https://github.com/pulsar-edit/pulsar.git
synced 2024-09-11 11:15:24 +03:00
Merge branch 'master' into migrate-deprecation-cop-package
This commit is contained in:
commit
fed020c382
2
.github/no-response.yml
vendored
2
.github/no-response.yml
vendored
@ -1,7 +1,7 @@
|
||||
# Configuration for probot-no-response - https://github.com/probot/no-response
|
||||
|
||||
# Number of days of inactivity before an issue is closed for lack of response
|
||||
daysUntilClose: 180
|
||||
daysUntilClose: 28
|
||||
|
||||
# Label requiring a response
|
||||
responseRequiredLabel: more-information-needed
|
||||
|
6
package-lock.json
generated
6
package-lock.json
generated
@ -1561,16 +1561,14 @@
|
||||
}
|
||||
},
|
||||
"dalek": {
|
||||
"version": "https://www.atom.io/api/packages/dalek/versions/0.2.2/tarball",
|
||||
"integrity": "sha512-7p7KFZFakk54JTKiP/gsXaEMAcsO30Y5dT3lrHZG3jeAMg201YSq5ayAd735i79S/RbpH32X5DaE6XC3Qf9BSw==",
|
||||
"version": "file:packages/dalek",
|
||||
"requires": {
|
||||
"grim": "^2.0.1"
|
||||
},
|
||||
"dependencies": {
|
||||
"grim": {
|
||||
"version": "2.0.2",
|
||||
"resolved": "https://registry.npmjs.org/grim/-/grim-2.0.2.tgz",
|
||||
"integrity": "sha1-52CinKe4NDsMH/r2ziDyGkbuiu0=",
|
||||
"bundled": true,
|
||||
"requires": {
|
||||
"event-kit": "^2.0.0"
|
||||
}
|
||||
|
@ -12,7 +12,7 @@
|
||||
"url": "https://github.com/atom/atom/issues"
|
||||
},
|
||||
"license": "MIT",
|
||||
"electronVersion": "2.0.10",
|
||||
"electronVersion": "2.0.11",
|
||||
"dependencies": {
|
||||
"@atom/nsfw": "1.0.18",
|
||||
"@atom/source-map-support": "^0.3.4",
|
||||
@ -47,7 +47,7 @@
|
||||
"coffee-script": "1.12.7",
|
||||
"color": "^0.7.3",
|
||||
"command-palette": "https://www.atom.io/api/packages/command-palette/versions/0.43.5/tarball",
|
||||
"dalek": "https://www.atom.io/api/packages/dalek/versions/0.2.2/tarball",
|
||||
"dalek": "file:packages/dalek",
|
||||
"dedent": "^0.7.0",
|
||||
"deprecation-cop": "file:packages/deprecation-cop",
|
||||
"dev-live-reload": "file:packages/dev-live-reload",
|
||||
@ -195,7 +195,7 @@
|
||||
"bookmarks": "0.45.1",
|
||||
"bracket-matcher": "0.89.3",
|
||||
"command-palette": "0.43.5",
|
||||
"dalek": "0.2.2",
|
||||
"dalek": "file:./packages/dalek",
|
||||
"deprecation-cop": "file:./packages/deprecation-cop",
|
||||
"dev-live-reload": "file:./packages/dev-live-reload",
|
||||
"encoding-selector": "0.23.9",
|
||||
|
@ -26,7 +26,7 @@ See [RFC 003](https://github.com/atom/atom/blob/master/docs/rfcs/003-consolidate
|
||||
| **bookmarks** | [`atom/bookmarks`][bookmarks] | |
|
||||
| **bracket-matcher** | [`atom/bracket-matcher`][bracket-matcher] | |
|
||||
| **command-palette** | [`atom/command-palette`][command-palette] | |
|
||||
| **dalek** | [`atom/dalek`][dalek] | [#17838](https://github.com/atom/atom/issues/17838) |
|
||||
| **dalek** | [`./dalek`](./dalek) | [#17838](https://github.com/atom/atom/issues/17838) |
|
||||
| **deprecation-cop** | [`./deprecation-cop`](./deprecation-cop) | [#17839](https://github.com/atom/atom/issues/17839) |
|
||||
| **dev-live-reload** | [`./dev-live-reload`](dev-live-reload) | [#17840](https://github.com/atom/atom/issues/17840) |
|
||||
| **encoding-selector** | [`atom/encoding-selector`][encoding-selector] | [#17841](https://github.com/atom/atom/issues/17841) |
|
||||
@ -113,7 +113,6 @@ See [RFC 003](https://github.com/atom/atom/blob/master/docs/rfcs/003-consolidate
|
||||
[bookmarks]: https://github.com/atom/bookmarks
|
||||
[bracket-matcher]: https://github.com/atom/bracket-matcher
|
||||
[command-palette]: https://github.com/atom/command-palette
|
||||
[dalek]: https://github.com/atom/dalek
|
||||
[encoding-selector]: https://github.com/atom/encoding-selector
|
||||
[find-and-replace]: https://github.com/atom/find-and-replace
|
||||
[fuzzy-finder]: https://github.com/atom/fuzzy-finder
|
||||
|
3
packages/dalek/.gitignore
vendored
Normal file
3
packages/dalek/.gitignore
vendored
Normal file
@ -0,0 +1,3 @@
|
||||
.DS_Store
|
||||
npm-debug.log
|
||||
node_modules
|
20
packages/dalek/LICENSE.md
Normal file
20
packages/dalek/LICENSE.md
Normal file
@ -0,0 +1,20 @@
|
||||
Copyright (c) 2016 GitHub, Inc.
|
||||
|
||||
Permission is hereby granted, free of charge, to any person obtaining
|
||||
a copy of this software and associated documentation files (the
|
||||
"Software"), to deal in the Software without restriction, including
|
||||
without limitation the rights to use, copy, modify, merge, publish,
|
||||
distribute, sublicense, and/or sell copies of the Software, and to
|
||||
permit persons to whom the Software is furnished to do so, subject to
|
||||
the following conditions:
|
||||
|
||||
The above copyright notice and this permission notice shall be
|
||||
included in all copies or substantial portions of the Software.
|
||||
|
||||
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
|
||||
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
||||
MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
|
||||
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
|
||||
LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
|
||||
OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
|
||||
WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
|
19
packages/dalek/README.md
Normal file
19
packages/dalek/README.md
Normal file
@ -0,0 +1,19 @@
|
||||
# dalek
|
||||
|
||||
**EXTERMINATEs** core packages installed in `~/.atom/packages`.
|
||||
|
||||
## Why worry?
|
||||
|
||||
When people install core Atom packages as if they are community packages, it can cause many problems that are very hard to diagnose. This package is intended to notify people when they are in this precarious position so they can take corrective action.
|
||||
|
||||
## I got a warning, what do I do?
|
||||
|
||||
1. Note down the packages named in the notification
|
||||
1. Exit Atom
|
||||
1. Open a command prompt
|
||||
1. For each package named in the notification, execute `apm uninstall [package-name]`
|
||||
1. Start Atom again normally to verify that the warning notification no longer appears
|
||||
|
||||
## I have more questions. Where can I ask them?
|
||||
|
||||
Please feel free to ask on [the official Atom message board](https://discuss.atom.io/c/support).
|
54
packages/dalek/lib/dalek.js
Normal file
54
packages/dalek/lib/dalek.js
Normal file
@ -0,0 +1,54 @@
|
||||
/** @babel */
|
||||
|
||||
const fs = require('fs')
|
||||
const path = require('path')
|
||||
|
||||
module.exports = {
|
||||
async enumerate () {
|
||||
if (atom.inDevMode()) {
|
||||
return []
|
||||
}
|
||||
|
||||
const duplicatePackages = []
|
||||
const names = atom.packages.getAvailablePackageNames()
|
||||
for (let name of names) {
|
||||
if (atom.packages.isBundledPackage(name)) {
|
||||
const isDuplicatedPackage = await this.isInstalledAsCommunityPackage(name)
|
||||
if (isDuplicatedPackage) {
|
||||
duplicatePackages.push(name)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return duplicatePackages
|
||||
},
|
||||
|
||||
async isInstalledAsCommunityPackage (name) {
|
||||
const availablePackagePaths = atom.packages.getPackageDirPaths()
|
||||
|
||||
for (let packagePath of availablePackagePaths) {
|
||||
const candidate = path.join(packagePath, name)
|
||||
|
||||
if (fs.existsSync(candidate)) {
|
||||
const realPath = await this.realpath(candidate)
|
||||
if (realPath === candidate) {
|
||||
return true
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return false
|
||||
},
|
||||
|
||||
realpath (path) {
|
||||
return new Promise((resolve, reject) => {
|
||||
fs.realpath(path, function (error, realpath) {
|
||||
if (error) {
|
||||
reject(error)
|
||||
} else {
|
||||
resolve(realpath)
|
||||
}
|
||||
})
|
||||
})
|
||||
}
|
||||
}
|
19
packages/dalek/lib/main.js
Normal file
19
packages/dalek/lib/main.js
Normal file
@ -0,0 +1,19 @@
|
||||
/** @babel */
|
||||
|
||||
const dalek = require('./dalek')
|
||||
const Grim = require('grim')
|
||||
|
||||
module.exports = {
|
||||
activate () {
|
||||
atom.packages.onDidActivateInitialPackages(async () => {
|
||||
const duplicates = await dalek.enumerate()
|
||||
for (let i = 0; i < duplicates.length; i++) {
|
||||
const duplicate = duplicates[i]
|
||||
Grim.deprecate(
|
||||
`You have the core package "${duplicate}" installed as a community package. See https://github.com/atom/atom/blob/master/packages/dalek/README.md for how this causes problems and instructions on how to correct the situation.`,
|
||||
{ packageName: duplicate }
|
||||
)
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
29
packages/dalek/package.json
Normal file
29
packages/dalek/package.json
Normal file
@ -0,0 +1,29 @@
|
||||
{
|
||||
"name": "dalek",
|
||||
"main": "./lib/main",
|
||||
"version": "0.2.2",
|
||||
"description": "EXTERMINATEs built-in packages installed in ~/.atom/packages",
|
||||
"keywords": [],
|
||||
"repository": "https://github.com/atom/atom",
|
||||
"license": "MIT",
|
||||
"atomTestRunner": "./test/runner",
|
||||
"engines": {
|
||||
"atom": ">=1.12.7 <2.0.0"
|
||||
},
|
||||
"dependencies": {
|
||||
"grim": "^2.0.1"
|
||||
},
|
||||
"devDependencies": {
|
||||
"atom-mocha-test-runner": "^1.0.0",
|
||||
"sinon": "^2.0.0",
|
||||
"standard": "^8.6.0"
|
||||
},
|
||||
"standard": {
|
||||
"env": {
|
||||
"jasmine": true
|
||||
},
|
||||
"globals": [
|
||||
"atom"
|
||||
]
|
||||
}
|
||||
}
|
66
packages/dalek/test/dalek.test.js
Normal file
66
packages/dalek/test/dalek.test.js
Normal file
@ -0,0 +1,66 @@
|
||||
/** @babel */
|
||||
|
||||
const assert = require('assert')
|
||||
const fs = require('fs')
|
||||
const sinon = require('sinon')
|
||||
const path = require('path')
|
||||
|
||||
const dalek = require('../lib/dalek')
|
||||
|
||||
describe('dalek', function () {
|
||||
describe('enumerate', function () {
|
||||
let availablePackages = {}
|
||||
let realPaths = {}
|
||||
let bundledPackages = []
|
||||
let packageDirPaths = []
|
||||
let sandbox = null
|
||||
|
||||
beforeEach(function () {
|
||||
availablePackages = {
|
||||
'an-unduplicated-installed-package': path.join('Users', 'username', '.atom', 'packages', 'an-unduplicated-installed-package'),
|
||||
'duplicated-package': path.join('Users', 'username', '.atom', 'packages', 'duplicated-package'),
|
||||
'unduplicated-package': path.join(`${atom.getLoadSettings().resourcePath}`, 'node_modules', 'unduplicated-package')
|
||||
}
|
||||
|
||||
atom.devMode = false
|
||||
bundledPackages = ['duplicated-package', 'unduplicated-package']
|
||||
packageDirPaths = [path.join('Users', 'username', '.atom', 'packages')]
|
||||
sandbox = sinon.sandbox.create()
|
||||
sandbox.stub(dalek, 'realpath').callsFake((filePath) => Promise.resolve(realPaths[filePath] || filePath))
|
||||
sandbox.stub(atom.packages, 'isBundledPackage').callsFake((packageName) => { return bundledPackages.includes(packageName) })
|
||||
sandbox.stub(atom.packages, 'getAvailablePackageNames').callsFake(() => Object.keys(availablePackages))
|
||||
sandbox.stub(atom.packages, 'getPackageDirPaths').callsFake(() => { return packageDirPaths })
|
||||
sandbox.stub(fs, 'existsSync').callsFake((candidate) => { return Object.values(availablePackages).includes(candidate) && !candidate.includes(atom.getLoadSettings().resourcePath) })
|
||||
})
|
||||
|
||||
afterEach(function () {
|
||||
sandbox.restore()
|
||||
})
|
||||
|
||||
it('returns a list of duplicate names', async function () {
|
||||
assert.deepEqual(await dalek.enumerate(), ['duplicated-package'])
|
||||
})
|
||||
|
||||
describe('when in dev mode', function () {
|
||||
beforeEach(function () {
|
||||
atom.devMode = true
|
||||
})
|
||||
|
||||
it('always returns an empty list', async function () {
|
||||
assert.deepEqual(await dalek.enumerate(), [])
|
||||
})
|
||||
})
|
||||
|
||||
describe('when a package is symlinked into the package directory', async function () {
|
||||
beforeEach(function () {
|
||||
const realPath = path.join('Users', 'username', 'duplicated-package')
|
||||
const packagePath = path.join('Users', 'username', '.atom', 'packages', 'duplicated-package')
|
||||
realPaths[packagePath] = realPath
|
||||
})
|
||||
|
||||
it('is not included in the list of duplicate names', async function () {
|
||||
assert.deepEqual(await dalek.enumerate(), [])
|
||||
})
|
||||
})
|
||||
})
|
||||
})
|
2
packages/dalek/test/runner.js
Normal file
2
packages/dalek/test/runner.js
Normal file
@ -0,0 +1,2 @@
|
||||
const createRunner = require('atom-mocha-test-runner').createRunner
|
||||
module.exports = createRunner({testSuffixes: ['test.js']})
|
Loading…
Reference in New Issue
Block a user