Skip to content

Commit

Permalink
Upgrade typescript to 3.8.2
Browse files Browse the repository at this point in the history
  • Loading branch information
ktmud committed Feb 24, 2020
1 parent 3efdfc5 commit 904ea19
Show file tree
Hide file tree
Showing 8 changed files with 13 additions and 11 deletions.
3 changes: 0 additions & 3 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -58,9 +58,6 @@ rat-results.txt
*.js.map
node_modules
npm-debug.log*
superset-frontend/coverage/*
superset-frontend/cypress/screenshots
superset-frontend/cypress/videos
superset/static/assets
superset/static/version_info.json
yarn-error.log
Expand Down
4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -488,15 +488,15 @@ best practices.

Superset supports a server-wide feature flag system, which eases the incremental development of features. To add a new feature flag, simply modify `superset_config.py` with something like the following:

```
```python
FEATURE_FLAGS = {
'SCOPED_FILTER': True,
}
```

If you want to use the same flag in the client code, also add it to the FeatureFlag TypeScript enum in `superset-frontend/src/featureFlags.ts`. For example,

```
```typescript
export enum FeatureFlag {
SCOPED_FILTER = 'SCOPED_FILTER',
}
Expand Down
1 change: 1 addition & 0 deletions superset-frontend/.eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,6 @@ stylesheets/*
vendor/*
docs/*
src/dashboard/deprecated/*
src/temp/*
**/node_modules
*.d.ts
4 changes: 4 additions & 0 deletions superset-frontend/.gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
coverage/*
cypress/screenshots
cypress/videos
src/temp
6 changes: 3 additions & 3 deletions superset-frontend/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion superset-frontend/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@
"ts-jest": "^24.0.2",
"ts-loader": "^5.4.5",
"tslib": "^1.10.0",
"typescript": "^3.5.3",
"typescript": "^3.8.2",
"url-loader": "^1.0.1",
"webpack": "^4.19.0",
"webpack-assets-manifest": "^3.0.1",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
import { safeStringify } from '../../../src/utils/safeStringify';

class Noise {
public next: Noise;
public next?: Noise;
}

describe('Stringify utility testing', () => {
Expand Down
2 changes: 1 addition & 1 deletion superset-frontend/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ const config = {
hot: true,
index: '', // This line is needed to enable root proxying
inline: true,
stats: { colors: true },
stats: 'minimal',
overlay: true,
port: devserverPort,
// Only serves bundled files from webpack-dev-server
Expand Down

0 comments on commit 904ea19

Please sign in to comment.