Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade project to typescript #69

Merged
merged 12 commits into from
Aug 21, 2020
Merged

upgrade project to typescript #69

merged 12 commits into from
Aug 21, 2020

Conversation

Debaerdm
Copy link
Contributor

Pull Request Template

PR Description

upgrade project to typescript

@Debaerdm Debaerdm self-assigned this Aug 12, 2020
@Debaerdm Debaerdm added component configuration dependencies Pull requests that update a dependency file svelte typescript labels Aug 12, 2020
@Debaerdm Debaerdm closed this Aug 19, 2020
@Debaerdm Debaerdm reopened this Aug 19, 2020
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 3 Code Smells

29.7% 29.7% Coverage
0.0% 0.0% Duplication

Copy link

@gui260586 gui260586 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Increase Code Coverage

@Debaerdm Debaerdm merged commit 5cf2b1c into master Aug 21, 2020
@Debaerdm Debaerdm deleted the feature/typescript branch August 21, 2020 14:21

const createPreprocessors = require('./svelte.config').createPreprocessors;
const production = !process.env.ROLLUP_WATCH;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ROLLUP_WATCH environment variable is about watch mode not production 😃 . You could use process.env.NODE_ENV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants