8000 automatically ignore tsc errors · teambit/bit-javascript@f11276d · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Dec 4, 2022. It is now read-only.

Commit f11276d

Browse files
author
David First
committed
automatically ignore tsc errors
1 parent ed72534 commit f11276d

File tree

10 files changed

+63
-0
lines changed

10 files changed

+63
-0
lines changed

‎src/cli/command-registrar.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ function logAndExit(str) {
3535
function start() {
3636
program.version(pkg.version).description('bit driver for javascript');
3737
commands.forEach(c => {
38+
// @ts-ignore FIXME
3839
const currentCommand = program.command(c.name).description(c.description);
3940

4041
if (c.options && Array.isArray(c.options)) {
@@ -48,6 +49,7 @@ function start() {
4849
c.action(args, options)
4950
.then(c.report)
5051
.then(logAndExit)
52+
// @ts-ignore FIXME
5153
.catch(c.handleError || errorHandler);
5254
});
5355
});

‎src/cli/loader/loader.ts‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,15 @@ const off = (): Loader | null | undefined => {
4646

4747
const loader: Loader = {
4848
on,
49+
// @ts-ignore FIXME
4950
off,
51+
// @ts-ignore FIXME
5052
stop,
53+
// @ts-ignore FIXME
5154
start,
55+
// @ts-ignore FIXME
5256
setText,
57+
// @ts-ignore FIXME
5358
get
5459
};
5560

‎src/dependency-builder/build-tree.spec.ts‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,19 +21,23 @@ describe('buildTree', () => {
2121
expect(results).to.deep.equal({ tree: {} });
2222
});
2323
it('when unsupported files are passed should return them with no dependencies', async () => {
24+
// @ts-ignore FIXME
2425
dependencyTreeParams.filePaths = [`${fixtures}/unsupported-file.pdf`];
2526
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
2627
const results = await buildTree.getDependencyTree(dependencyTreeParams);
2728
expect(results.tree).to.deep.equal({ 'fixtures/unsupported-file.pdf': {} });
2829
});
2930
it('when supported and unsupported files are passed should return them all', async () => {
31+
// @ts-ignore FIXME
32+
// @ts-ignore FIXME
3033
dependencyTreeParams.filePaths = [`${fixtures}/unsupported-file.pdf`, `${precinctFixtures}/es6.js`];
3134
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
3235
const results = await buildTree.getDependencyTree(dependencyTreeParams);
3336
expect(results.tree).to.have.property('fixtures/unsupported-file.pdf');
3437
expect(results.tree).to.have.property('fixtures/precinct/es6.js');
3538
});
3639
it('when a js file has parsing error it should add the file to the tree with the error instance', async () => {
40+
// @ts-ignore FIXME
3741
dependencyTreeParams.filePaths = [`${precinctFixtures}/unparseable.js`];
3842
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
3943
const results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -43,6 +47,7 @@ describe('buildTree', () => {
4347
expect(results.tree[unParsedFile].error).to.be.instanceof(Error);
4448
});
4549
it('when a js file has parsing error and it retrieved from the cache it should add the file to the tree with the error instance', async () => {
50+
// @ts-ignore FIXME
4651
dependencyTreeParams.filePaths = [`${precinctFixtures}/unparseable.js`];
4752
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
4853
dependencyTreeParams.visited = {};
@@ -61,6 +66,7 @@ describe('buildTree', () => {
6166
expect(resultsCached.tree[unParsedFile].error).to.be.instanceof(Error);
6267
});
6368
it.skip('when a css file has parsing error it should add the file to the tree with the error instance', async () => {
69+
// @ts-ignore FIXME
6470
dependencyTreeParams.filePaths = [`${buildTreeFixtures}/unparsed.css`];
6571
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
6672
const results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -72,6 +78,8 @@ describe('buildTree', () => {
7278
describe('when a dependency of dependency has parsing error', () => {
7379
let results;
7480
before(async () => {
81+
// @ts-ignore FIXME
82+
// @ts-ignore FIXME
7583
dependencyTreeParams.filePaths = [`${buildTreeFixtures}/a.js`, `${buildTreeFixtures}/b.js`];
7684
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
7785
results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -94,6 +102,7 @@ describe('buildTree', () => {
94102
let results;
95103
const missingDepsFile = 'fixtures/missing-deps.js';
96104
before(async () => {
105+
// @ts-ignore FIXME
97106
dependencyTreeParams.filePaths = [`${fixtures}/missing-deps.js`];
98107
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
99108
results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -113,6 +122,7 @@ describe('buildTree', () => {
113122
describe('when a file imports from itself', () => {
114123
let results;
115124
before(async () => {
125+
// @ts-ignore FIXME
116126
dependencyTreeParams.filePaths = [`${buildTreeFixtures}/tree-shaking-cycle/self-cycle.js`];
117127
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
118128
results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -125,6 +135,7 @@ describe('buildTree', () => {
125135
describe('cycle with multiple files', () => {
126136
let results;
127137
before(async () => {
138+
// @ts-ignore FIXME
128139
dependencyTreeParams.filePaths = [`${buildTreeFixtures}/tree-shaking-cycle/foo.js`];
129140
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
130141
results = await buildTree.getDependencyTree(dependencyTreeParams);
@@ -144,6 +155,7 @@ describe('buildTree', () => {
144155
describe('fileA imports varX from fileB, fileB imports varX from fileC but not export it', () => {
145156
let results;
146157
before(async () => {
158+
// @ts-ignore FIXME
147159
dependencyTreeParams.filePaths = [`${buildTreeFixtures}/not-link-file/file-a.js`];
148160
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
149161
results = await buildTree.getDependencyTree(dependencyTreeParams);

‎src/dependency-builder/build-tree.ts‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,12 @@ export function resolveNodePackage(cwd: string, packageFullPath: string): Record
8282
// if you have 2 authored component which one dependet on the other
8383
// we will look for the package.json on the dependency but won't find it
8484
// if we propagate we will take the 103C7 version from the root's package json which has nothing with the component version
85+
// @ts-ignore FIXME
8586
const packageInfo = PackageJson.loadSync(packageDir, false);
8687

8788
// when running 'bitjs get-dependencies' command, packageInfo is sometimes empty
8889
// or when using custom-module-resolution it may be empty or the name/version are empty
90+
// @ts-ignore FIXME
8991
if (!packageInfo || !packageInfo.name || !packageInfo.version) return null;
9092
result[packageInfo.name] = packageInfo.version;
9193
return result;
@@ -129,6 +131,7 @@ function groupDependencyList(list, cwd, bindingPrefix) {
129131
groups.packages.forEach(packagePath => {
130132
const packageWithVersion = resolveNodePackage(cwd, path.join(cwd, packagePath));
131133
if (packageWithVersion) Object.assign(packages, packageWithVersion);
134+
// @ts-ignore FIXME
132135
else unidentifiedPackages.push(packagePath);
133136
});
134137
groups.packages = packages;
@@ -259,6 +262,7 @@ function groupMissing(missing, cwd, consumerPath, bindingPrefix) {
259262
if (R.contains(packageName, missingPackages)) return;
260263
const resolvedPath = resolveModulePath(packageName, cwd, consumerPath);
261264
if (!resolvedPath) {
265+
// @ts-ignore FIXME
262266
missingPackages.push(packageName);
263267
return;
264268
}
@@ -413,6 +417,7 @@ export async function getDependencyTree({
413417
resolveConfig: resolveConfigAbsolute,
414418
cacheProjectAst
415419
};
420+
// @ts-ignore FIXME
416421
const { madgeTree, skipped, pathMap, errors } = generateTree(filePaths, config);
417422
const tree: Tree = groupDependencyTree(madgeTree, baseDir, bindingPrefix);
418423
const { missingGroups, foundPackages } = groupMissing(skipped, baseDir, consumerPath, bindingPrefix);

‎src/dependency-builder/dependency-tree/Config.ts‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,36 +9,58 @@ const debug = require('debug')('tree');
99

1010
// @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
1111
function Config(options) {
12+
// @ts-ignore FIXME
1213
this.filename = options.filename;
14+
// @ts-ignore FIXME
1315
this.directory = options.directory || options.root;
16+
// @ts-ignore FIXME
1417
this.visited = options.visited || {};
18+
// @ts-ignore FIXME
1519
this.errors = options.errors || {};
20+
// @ts-ignore FIXME
1621
this.nonExistent = options.nonExistent || [];
22+
// @ts-ignore FIXME
1723
this.isListForm = options.isListForm;
24+
// @ts-ignore FIXME
1825
this.requireConfig = options.config || options.requireConfig;
26+
// @ts-ignore FIXME
1927
this.webpackConfig = options.webpackConfig;
28+
// @ts-ignore FIXME
2029
this.detectiveConfig = options.detective || options.detectiveConfig || {};
30+
// @ts-ignore FIXME
2131
this.pathMap = options.pathMap || [];
32+
// @ts-ignore FIXME
2233
this.resolveConfig = options.resolveConfig;
34+
// @ts-ignore FIXME
2335
this.cacheProjectAst = options.cacheProjectAst;
2436

37+
// @ts-ignore FIXME
2538
this.filter = options.filter;
2639

40+
// @ts-ignore FIXME
2741
if (!this.filename) {
2842
throw new Error('filename not given');
2943
}
44+
// @ts-ignore FIXME
3045
if (!this.directory) {
3146
throw new Error('directory not given');
3247
}
48+
// @ts-ignore FIXME
49+
// @ts-ignore FIXME
3350
if (this.filter && typeof this.filter !== 'function') {
3451
throw new Error('filter must be a function');
3552
}
3653

54+
// @ts-ignore FIXME
3755
debug(`given filename: ${this.filename}`);
3856

57+
// @ts-ignore FIXME
58+
// @ts-ignore FIXME
3959
this.filename = path.resolve(process.cwd(), this.filename);
4060

61+
// @ts-ignore FIXME
4162
debug(`resolved filename: ${this.filename}`);
63+
// @ts-ignore FIXME
4264
debug('visited: ', this.visited);
4365
}
4466

‎src/dependency-builder/dependency-tree/index.spec.ts‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -963,8 +963,11 @@ describe('dependencyTree', function() {
963963
resolveConfig: { aliases: { something: 'anything' } }
964964
};
965965
dependencyTree(config);
966+
// @ts-ignore FIXME
966967
const pathMapRecord = config.pathMap.find(f => f.file === filename);
968+
// @ts-ignore FIXME
967969
expect(pathMapRecord.dependencies).to.have.lengthOf(1);
970+
// @ts-ignore FIXME
968971
const dependency = pathMapRecord.dependencies[0];
969972
expect(dependency).to.not.have.property('isCustomResolveUsed');
970973
});
@@ -988,8 +991,11 @@ describe('dependencyTree', function() {
988991
resolveConfig: { aliases: { something: 'anything' } }
989992
};
990993
dependencyTree(config);
994+
// @ts-ignore FIXME
991995
const pathMapRecord = config.pathMap.find(f => f.file === filename);
996+
// @ts-ignore FIXME
992997
expect(pathMapRecord.dependencies).to.have.lengthOf(1);
998+
// @ts-ignore FIXME
993999
const dependency = pathMapRecord.dependencies[0];
9941000
expect(dependency).to.not.have.property('isCustomResolveUsed');
9951001
});

‎src/dependency-builder/dependency-tree/index.ts‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,10 @@ module.exports._getDependencies = function(config) {
147147
pathMap.isCustomResolveUsed = true;
148148
}
149149

150+
// @ts-ignore FIXME
150151
pathMapDependencies.push(pathMap);
151152

153+
// @ts-ignore FIXME
152154
resolvedDependencies.push(result);
153155
}
154156
function addToNonExistent(dependency) {
@@ -195,6 +197,7 @@ function traverse(config) {
195197
debug(`number of dependencies after filtering: ${dependencies.length}`);
196198
}
197199
debug('cabinet-resolved all dependencies: ', dependencies);
200+
// @ts-ignore FIXME
198201
tree[dependency] = dependencies;
199202
const filePathMap = config.pathMap.find(pathMapEntry => pathMapEntry.file === dependency);
200203
if (!filePathMap) throw new Error(`file ${dependency} is missing from PathMap`);
@@ -222,6 +225,7 @@ function traverse(config) {
222225
}
223226
debug(`found ${dependency} in the cache`);
224227
const dependencies = config.visited[dependency].pathMap.dependencies.map(d => d.resolvedDep);
228+
// @ts-ignore FIXME
225229
tree[dependency] = dependencies;
226230
config.pathMap.push(config.visited[dependency].pathMap);
227231
if (config.visited[dependency].missing) {

‎src/dependency-builder/filing-cabinet/index.spec.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,7 @@ describe('filing-cabinet', () => {
211211
});
212212

213213
assert.ok(
214+
// @ts-ignore FIXME
214215
require.main.paths.some(function(p) {
215216
return p.indexOf(path.normalize(directory)) !== -1;
216217
})

‎src/dependency-builder/generate-tree-madge.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,7 @@ export default function generateTree(files = [], config) {
147147
});
148148
Object.assign(depTree, dependencyTreeResult);
149149
} catch (err) {
150+
// @ts-ignore FIXME
150151
errors[file] = err;
151152
}
152153
});

‎src/dependency-builder/path-map.ts‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,16 @@ function getDependenciesFromLinkFileIfExists(
145145
return null;
146146
}
147147
const linkFiles = [];
148+
// @ts-ignore FIXME
148149
dependencies.forEach((dep: { file: string; importSpecifier: ImportSpecifier }) => {
150+
// @ts-ignore FIXME
149151
const existingFile = linkFiles.find(linkFile => linkFile.file === dep.file);
150152
if (existingFile) {
153+
// @ts-ignore FIXME
151154
existingFile.importSpecifiers.push(dep.importSpecifier);
152155
} else {
156+
// @ts-ignore FIXME
157+
// @ts-ignore FIXME
153158
linkFiles.push({ file: dep.file, importSpecifiers: [dep.importSpecifier] });
154159
}
155160
});

0 commit comments

Comments
 (0)
0