GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add --skipLibCheck compiler option by ahejlsberg · Pull Request #8735 · microsoft/TypeScript

Conversation

@ahejlsberg

@ahejlsberg Anders Hejlsberg (ahejlsberg) commented May 22, 2016 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

This PR adds a new --skipLibCheck compiler option that causes type checking of declaration files (files with extension .d.ts) to be skipped. When a program includes large declaration files, the compiler spends a lot of time type checking declarations that are already known to not contain errors, and compile times may be significantly shortened by skipping declaration file type checks.

Since declarations in one file can affect type checking in other files, some errors may not be detected when --skipLibCheck is specified. For example, if a non-declaration file augments a type declared in a declaration file, errors may result that are only reported when the declaration file is checked. However, in practice such situations are rare.

Fixes #8521. Compile times for the example given in this issue are reduced 5x.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Mr-Byte, SergioMorchon, WanderWang, christopherthielen, styfle, Jimeux, mrfyda, and b0bi79 reacted with thumbs up emoji
All reactions
  • 👍 8 reactions
@ahejlsberg
Add --skipLibCheck compiler option
db109d1
@msftclas Microsoft Contribution License Agreements (msftclas) added the cla-not-required label May 22, 2016
@DanielRosenwasser

Daniel Rosenwasser (DanielRosenwasser) commented May 22, 2016

Copy link
Copy Markdown
Member

👍

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

2 similar comments
@mhegazy

Mohamed Hegazy (mhegazy) commented May 23, 2016

Copy link
Copy Markdown
Contributor

👍

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@vladima

Vladimir Matveev (vladima) commented May 23, 2016

Copy link
Copy Markdown
Contributor

👍

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@ahejlsberg
Merge branch 'master' into skipLibCheck
f322def
@ahejlsberg Anders Hejlsberg (ahejlsberg) closed this May 23, 2016
@ahejlsberg
Anders Hejlsberg (ahejlsberg) deleted the skipLibCheck branch May 23, 2016 18:02
@johnnyreilly John Reilly (johnnyreilly) mentioned this pull request May 30, 2016
noImplicitAny rule applied to external packages #8883
Closed
@christopherthielen Chris Thielen (christopherthielen) mentioned this pull request Jun 24, 2016
Alpha.4 has many TS7006 error "implicitly has an any type" angular-ui/ui-router#2693
Closed
@mhegazy Mohamed Hegazy (mhegazy) mentioned this pull request Jun 30, 2016
Ability to over power TS and ignore specific error by developer #9448
Closed
@DanielRosenwasser Daniel Rosenwasser (DanielRosenwasser) mentioned this pull request Jul 7, 2016
TypeScript support amireh/happypack#56
Closed
7 of 17 tasks
@saschanaz Kagami Sascha Rosylight (saschanaz) mentioned this pull request Jul 26, 2016
Add an option to skip ///<reference> check on d.ts files #9945
Closed
basarat
basarat reviewed Jul 27, 2016
View reviewed changes
Comment thread src/compiler/checker.ts
}
// If skipLibCheck is enabled, skip type checking if file is a declaration file.
// If skipDefaultLibCheck is enabled, skip type checking if file contains a
// '/// <reference no-default-lib="true"/>' directive.

@basarat Basarat Ali Syed (basarat) Jul 27, 2016

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Curious why we would skip for noDefaultLib. Is this to support just test harnesses or actual user code? 🌹

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@microsoft Microsoft (microsoft) locked and limited conversation to collaborators Jun 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

6 participants

@ahejlsberg @DanielRosenwasser @mhegazy @vladima @basarat @msftclas
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.