Skip to content

Commit

Permalink
Remove trailing whitespace
Browse files Browse the repository at this point in the history
  • Loading branch information
malept committed Sep 8, 2018
1 parent 2bab74c commit 4a1c449
Show file tree
Hide file tree
Showing 22 changed files with 56 additions and 58 deletions.
2 changes: 1 addition & 1 deletion packages/api/cli/src/electron-forge-install.ts
Expand Up @@ -16,7 +16,7 @@ import './util/terminate';
repo = repository;
})
.parse(process.argv);

const chooseAsset = async (assets: InstallAsset[]) => {
const choices: { name: string, value: string }[] = [];
assets.forEach((asset) => {
Expand Down
8 changes: 4 additions & 4 deletions packages/api/core/src/api/index.ts
Expand Up @@ -16,7 +16,7 @@ import { fromBuildIdentifier } from '../util/forge-config';
export class ForgeAPI {
/**
* Attempt to import a given module directory to the Electron Forge standard.
*
*
* * Sets up `git` and the correct NPM dependencies
* * Adds a template forge config to `package.json`
*/
Expand All @@ -31,15 +31,15 @@ export class ForgeAPI {
}
/**
* Install an Electron application from GitHub.
*
*
* Works on all three platforms for all major distributable types.
*/
install(opts: InstallOptions) {
return install(opts);
}
/**
* Lint a local Electron application.
*
*
* The promise will be rejected with the stdout+stderr of the linting process
* if linting fails or will be resolved if it succeeds.
*/
Expand All @@ -66,7 +66,7 @@ export class ForgeAPI {
}
/**
* Start an Electron application.
*
*
* Handles things like native module rebuilding for you on the fly
*/
start(opts: StartOptions) {
Expand Down
4 changes: 2 additions & 2 deletions packages/api/core/src/api/publish.ts
Expand Up @@ -45,7 +45,7 @@ export interface PublishOptions {
dryRun?: boolean;
/**
* Whether or not to attempt to resume a previously saved `dryRun` and publish
*
*
* You can't use this combination at the same time as dryRun=true
*/
dryRunResume?: boolean;
Expand Down Expand Up @@ -97,7 +97,7 @@ const publish = async ({
}
return;
}

if (!makeResults) {
d('triggering make');
makeResults = await make(Object.assign({
Expand Down
2 changes: 1 addition & 1 deletion packages/api/core/src/util/plugin-interface.ts
Expand Up @@ -17,7 +17,7 @@ export default class PluginInterface implements IForgePluginInterface {
if ((plugin as IForgePlugin).__isElectronForgePlugin) {
return plugin;
}

if (Array.isArray(plugin)) {
if (typeof plugin[0] !== 'string') {
throw `Expected plugin[0] to be a string but found ${plugin[0]}`;
Expand Down
2 changes: 1 addition & 1 deletion packages/api/core/src/util/read-package-json.ts
Expand Up @@ -7,5 +7,5 @@ import { runMutatingHook } from './hook';
export const readRawPackageJson = async (dir: string) =>
await fs.readJson(path.resolve(dir, 'package.json'));

export const readMutatedPackageJson = async (dir: string, forgeConfig: ForgeConfig) =>
export const readMutatedPackageJson = async (dir: string, forgeConfig: ForgeConfig) =>
runMutatingHook(forgeConfig, 'readPackageJson', await readRawPackageJson(dir));
1 change: 0 additions & 1 deletion packages/api/core/test/fast/forge-config_spec.ts
Expand Up @@ -50,7 +50,6 @@ describe('forge-config', () => {
delete process.env.ELECTRON_FORGE_S3_SECRET_ACCESS_KEY;
});


it('should resolve the JS file exports in config.forge points to a JS file', async () => {
const config = JSON.parse(JSON.stringify(await findConfig(path.resolve(__dirname, '../fixture/dummy_js_conf'))));
delete config.pluginInterface;
Expand Down
4 changes: 2 additions & 2 deletions packages/api/core/test/fast/hook_spec.ts
Expand Up @@ -16,11 +16,11 @@ describe('hooks', () => {
it('should not error when running non existent hooks', async () => {
await runHook(Object.assign({}, fakeConfig), 'magic');
});

it('should not error when running a hook that is not a function', async () => {
await runHook(Object.assign({ hooks: { myHook: 'abc' } }, fakeConfig), 'abc');
});

it('should run the hook if it is provided as a function', async () => {
const myStub = stub();
myStub.returns(Promise.resolve());
Expand Down
1 change: 0 additions & 1 deletion packages/maker/base/src/Maker.ts
Expand Up @@ -90,7 +90,6 @@ export default abstract class Maker<C> {
return [];
}


/**
* Helpers
*/
Expand Down
18 changes: 9 additions & 9 deletions packages/maker/deb/src/Config.ts
Expand Up @@ -2,9 +2,9 @@ export interface MakerDebConfigOptions {
/**
* Name of the package (e.g. atom), used in the Package field of the control
* specification.
*
*
* Package names [...] must consist only of lower case letters (a-z), digits
* (0-9), plus (+) and minus (-) signs, and periods (.). They must be at
* (0-9), plus (+) and minus (-) signs, and periods (.). They must be at
* least two characters long and must start with an alphanumeric character.
*/
name?: string;
Expand Down Expand Up @@ -34,15 +34,15 @@ export interface MakerDebConfigOptions {
revision?: string;
/**
* Application area into which the package has been classified.
*
*
* Possible sections. Generated on https://packages.debian.org/unstable/ with:
*
*
* $$('#content dt a').map(n => {const ss = n.href.split('/'); return `'${ss[ss.length - 2]}'`; }).sort().join(' | ')
*/
section?: 'admin' | 'cli-mono' | 'comm' | 'database' | 'debian-installer' | 'debug' | 'devel' | 'doc' | 'editors' | 'education' | 'electronics' | 'embedded' | 'fonts' | 'games' | 'gnome' | 'gnu-r' | 'gnustep' | 'graphics' | 'hamradio' | 'haskell' | 'httpd' | 'interpreters' | 'introspection' | 'java' | 'javascript' | 'kde' | 'kernel' | 'libdevel' | 'libs' | 'lisp' | 'localization' | 'mail' | 'math' | 'metapackages' | 'misc' | 'net' | 'news' | 'ocaml' | 'oldlibs' | 'otherosfs' | 'perl' | 'php' | 'python' | 'ruby' | 'rust' | 'science' | 'shells' | 'sound' | 'tasks' | 'tex' | 'text' | 'utils' | 'vcs' | 'video' | 'virtual' | 'web' | 'x11' | 'xfce' | 'zope';
/**
* How important is it to have the package installed.
*
*
* You can read more: https://www.debian.org/doc/debian-policy/#priorities
*/
priority?: 'required' | 'important' | 'standard' | 'optional';
Expand Down Expand Up @@ -81,7 +81,7 @@ export interface MakerDebConfigOptions {
homepage?: string;
/**
* Relative path to the executable that will act as binary for the application, used in the Exec field of the desktop specification.
*
*
* Defaults to options.name
*/
bin?: string;
Expand All @@ -91,9 +91,9 @@ export interface MakerDebConfigOptions {
icon?: string;
/**
* Categories in which the application should be shown in a menu, used in the Categories field of the desktop specification.
*
*
* Generated on https://specifications.freedesktop.org/menu-spec/latest/apa.html with:
*
*
* `(${$$('.informaltable tr td:first-child').map(td => `'${td.innerText}'`).join(' | ')})[]`
*/
categories?: ('AudioVideo' | 'Audio' | 'Video' | 'Development' | 'Education' | 'Game' | 'Graphics' | 'Network' | 'Office' | 'Science' | 'Settings' | 'System' | 'Utility')[];
Expand All @@ -107,7 +107,7 @@ export interface MakerDebConfigOptions {
lintianOverrides?: string[];
/**
* Path to package maintainer scripts with their corresponding name, used in the installation procedure:
*
*
* Read More: https://www.debian.org/doc/debian-policy/#package-maintainer-scripts-and-installation-procedure
*/
scripts?: {
Expand Down
2 changes: 1 addition & 1 deletion packages/maker/dmg/src/Config.ts
Expand Up @@ -60,7 +60,7 @@ export interface MakerDMGConfig {
iconSize?: number;
/**
* Disk image format
*
*
* Default: UDZO
*/
format?: 'UDRW' | 'UDRO' | 'UDCO' | 'UDZO' | 'UDBZ' | 'ULFO';
Expand Down
20 changes: 10 additions & 10 deletions packages/maker/flatpak/src/Config.ts
@@ -1,7 +1,7 @@
export interface MakerFlatpakOptionsConfig {
/**
* App id of the flatpak, used in the id field of a flatpak-builder manifest.
*
*
* Default: io.atom.electron
*/
id?: string;
Expand All @@ -19,19 +19,19 @@ export interface MakerFlatpakOptionsConfig {
description?: string;
/**
* Release branch of the flatpak, used in the branch field of a flatpak-builder manifest.
*
*
* Default: master
*/
branch?: string;
/**
* Base app to use when building the flatpak, used in the base field of a flatpak-builder manifest.
*
*
* Default: io.atom.electron.BaseApp
*/
base?: string;
/**
* Base app version, used in the base-version field of a flatpak-builder manifest.
*
*
* Default: master
*/
baseVersion?: string;
Expand All @@ -41,19 +41,19 @@ export interface MakerFlatpakOptionsConfig {
baseFlatpakref?: string;
/**
* Runtime id, used in the runtime field of a flatpak-builder manifest.
*
*
* Default: org.freedesktop.Platform
*/
runtime?: string;
/**
* Runtime version, used in the runtime-version field of a flatpak-builder manifest.
*
*
* Default: 1.4
*/
runtimeVersion?: string;
/**
* Sdk id, used in the sdk field of a flatpak-builder manifest.
*
*
* Default: org.freedesktop.Sdk
*/
sdk?: string;
Expand All @@ -66,7 +66,7 @@ export interface MakerFlatpakOptionsConfig {
* Source should be a relative/absolute path to a file/directory to copy
* into the flatpak, and dest should be the path inside the app install
* prefix (e.g. /share/applications/)
*
*
* Application assets and code will be fully handled by electron-packager,
* but this is a useful way to install things such as appstream metadata
* for an app, or dbus configuration files.
Expand All @@ -89,9 +89,9 @@ export interface MakerFlatpakOptionsConfig {
icon?: string;
/**
* Categories in which the application should be shown in a menu, used in the Categories field of the desktop specification.
*
*
* Generated on https://specifications.freedesktop.org/menu-spec/latest/apa.html with:
*
*
* `(${$$('.informaltable tr td:first-child').map(td => `'${td.innerText}'`).join(' | ')})[]`
*/
categories?: ('AudioVideo' | 'Audio' | 'Video' | 'Development' | 'Education' | 'Game' | 'Graphics' | 'Network' | 'Office' | 'Science' | 'Settings' | 'System' | 'Utility')[];
Expand Down
6 changes: 3 additions & 3 deletions packages/maker/pkg/src/Config.ts
@@ -1,15 +1,15 @@
export interface MakerPKGConfig {
/**
* Name of certificate to use when signing.
*
*
* Default to be selected with respect to platform from keychain or keychain
* by system default.
*/
identity?: string;
/**
* Flag to enable/disable validation for signing identity. If enabled, the
* identity provided will be validated in the keychain specified.
*
*
* Default: `true`.
*/
'identity-validation'?: boolean;
Expand All @@ -19,7 +19,7 @@ export interface MakerPKGConfig {
install?: string;
/**
* The keychain name.
*
*
* Default: System default keychain.
*/
keychain?: string;
Expand Down
6 changes: 3 additions & 3 deletions packages/maker/rpm/src/Config.ts
Expand Up @@ -50,7 +50,7 @@ export interface MakerRpmConfigOptions {
compressionLevel?: 0 | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9;
/**
* Relative path to the executable that will act as binary for the application, used in the Exec field of the desktop specification.
*
*
* Defaults to options.name
*/
bin?: string;
Expand All @@ -64,9 +64,9 @@ export interface MakerRpmConfigOptions {
icon?: string;
/**
* Categories in which the application should be shown in a menu, used in the Categories field of the desktop specification.
*
*
* Generated on https://specifications.freedesktop.org/menu-spec/latest/apa.html with:
*
*
* `(${$$('.informaltable tr td:first-child').map(td => `'${td.innerText}'`).join(' | ')})[]`
*/
categories?: ('AudioVideo' | 'Audio' | 'Video' | 'Development' | 'Education' | 'Game' | 'Graphics' | 'Network' | 'Office' | 'Science' | 'Settings' | 'System' | 'Utility')[];
Expand Down
4 changes: 2 additions & 2 deletions packages/maker/snap/src/Config.ts
Expand Up @@ -21,7 +21,7 @@ export interface MakerSnapConfig {
appSlots?: string[];
/**
* See the [Snapcraft documentation](https://snapcraft.io/docs/reference/confinement).
*
*
* Default: devmode
*/
confinement?: 'strict' | 'devmode' | 'classic';
Expand Down Expand Up @@ -57,7 +57,7 @@ export interface MakerSnapConfig {
browserSandbox?: true;
/**
* [MPRIS](https://specifications.freedesktop.org/mpris-spec/latest/) support.
*
*
* If enabled, the interface name must be specified as the feature value.
*/
mpris?: string;
Expand Down
Expand Up @@ -47,7 +47,7 @@ describe('LocalElectronPlugin', () => {

describe('hooks', () => {
let p: LocalElectronPlugin;

beforeEach(() => {
p = new LocalElectronPlugin({ electronPath: 'test/foo' });
});
Expand Down
2 changes: 1 addition & 1 deletion packages/publisher/base/src/Publisher.ts
Expand Up @@ -13,7 +13,7 @@ export interface PublisherOptions {
makeResults: ForgeMakeResult[];
/**
* The raw forgeConfig this app is using.
*
*
* You probably shouldn't use this
*/
forgeConfig: ForgeConfig;
Expand Down
6 changes: 3 additions & 3 deletions packages/publisher/electron-release-server/src/Config.ts
@@ -1,7 +1,7 @@
export interface PublisherERSConfig {
/**
* The base URL of your instance of ERS.
*
*
* E.g. https://my-update.server.com
*/
baseUrl: string;
Expand All @@ -16,10 +16,10 @@ export interface PublisherERSConfig {
/**
* The release channel you want to send artifacts to, normally something like
* "stable", "beta" or "alpha".
*
*
* If left unspecified we will try to infer the channel from your version
* field in your package.json.
*
*
* Default: stable
*/
channel?: string;
Expand Down
2 changes: 1 addition & 1 deletion packages/publisher/github/src/Config.ts
Expand Up @@ -20,7 +20,7 @@ export interface PublisherGitHubConfig {
/**
* An authorization token with permission to upload releases to this
* repository.
*
*
* You can set the GITHUB_TOKEN environment variable if you don't want to hard
* code this into your config.
*/
Expand Down
4 changes: 2 additions & 2 deletions packages/publisher/github/src/PublisherGithub.ts
Expand Up @@ -73,7 +73,7 @@ export default class PublisherGithub extends PublisherBase<PublisherGitHubConfig
}
}
});

let uploaded = 0;
await asyncOra(`Uploading Artifacts ${uploaded}/${artifacts.length} to v${releaseName}`, async (uploadSpinner) => {
const updateSpinner = () => {
Expand All @@ -84,7 +84,7 @@ export default class PublisherGithub extends PublisherBase<PublisherGitHubConfig
for (const artifact of artifacts) {
flatArtifacts.push(...artifact.artifacts);
}

await Promise.all(flatArtifacts.map(artifactPath =>
new Promise(async (resolve) => {
const done = () => {
Expand Down

0 comments on commit 4a1c449

Please sign in to comment.