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.

Sign up for GitHub

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

Jump to bottom

Compilation of Js Files #4826

Closed
wants to merge 78 commits into from
Closed

Compilation of Js Files #4826

wants to merge 78 commits into from

Conversation

sheetalkamat
Copy link
Member

Compiler treats javascript files like declaration files and does not emit any output corresponding to them. Here are the list of work this PR supports:

  • Allow javascript files to be compiled on command line or through references/imports
  • tsconfig.json can take javascript files as file names to be compiled
  • If filenames are not specified in tsconfig.json, javascript files in the folder and not part of compilation
  • compiler options now have a flag named "consumeJsFiles" which when true will pick up javascript files in the folder for compilation if tscconfig.json doesn't supply file name list
  • Moved reporting of usage of typescript syntax in javascript file from services to compiler
  • Reports error if output is overwriting any of the input files
  • Reports error if output is not unique location and multiple input files overwrite same output file
  • It is error to reference .js file in a typescript file when generating declaration file.
  • .ts, .tsx and .d.ts files take precedence over .js files when creating list of files for compilation
  • Project runner options cleanup
  • Project runner to accept tsconfig.json tests
  • Instead of consumeJsFiles option, support jsExtensions that allows user to specify comma separated list of extensions that are to be treated like javascript files
  • Bundle js files into --out and copy into --outDir

@sheetalkamat
Add .js as supported extension
0a67ac3
@sheetalkamat
Emit the diagnostics for javascript file instead of doing semantic check
74a3f67
@sheetalkamat
Baseline updates
279018d
@sheetalkamat
Allow the js files to emit output
e9688dd
@sheetalkamat
Report error if output file is among the input files
fad0db2
@sheetalkamat
Add test cases for simple js file compilations
d3dfd2a
@sheetalkamat
Javascript files will emit declarations too so get declaration diagno…
1dc3414 
…stics
@sheetalkamat
Let harness to name file even when single unit data specified but the…
63de162 
…re is filename tag
@sheetalkamat
Test cases for different typescript syntax not supported errors durin…
57e17d2 
…g js files compilation
@sheetalkamat
Test for rest parameters(copied from language service tests)
7e30827
@sheetalkamat
Verify syntax error in js file are reported
2252354
@sheetalkamat
Let tsconfig to pick up js files
3107bbb
@sheetalkamat
Merge branch 'master' into tscJsFiles
885babc
@sheetalkamat
Cleanup of options of project runner
60f295f
@sheetalkamat
Project testcase to run tsconfig file
8da3bd2
@sheetalkamat
Add tests when same named .ts and .js file exist with tsconfig file s…
8aeff92 
…pecifying .ts file
@sheetalkamat
Merge branch 'master' into tscJsFiles
bc48c7c
@sheetalkamat
Verify and fix scenario when .js and .ts files with same name are pre…
9daf635 
…sent and tsconfig doesnt specify any filenames
@sheetalkamat
When same named .d.ts and .js files are present and tscconfig contain…
70d3de4 
…s .d.ts file
@sheetalkamat
When folder contains .d.ts as well as .js of same name and tsconfig d…
0f73c16 
…oesnt contain any name
@sheetalkamat
Tests when the .ts and .js files are mixed in compilation with tsccon…
dbb2772 
…fig file specifying them
@sheetalkamat
Tests when the .ts and .js files are mixed in compilation with tsccon…
14b6082 
…fig file doesnt specifying any names
@sheetalkamat
Do not emit javascript files
2860435
@sheetalkamat
Test case when one of the input file is output file name
68c65cd
@sheetalkamat
When tsconfig file doesnt contain file names, consume .js files of di…
fce9f32 
…rectory only if specified in the options
@sheetalkamat
Corrected scenario names in the test cases
e32c920
@sheetalkamat
Report error when emitting declarations if the reference is to .js file
60e15b2
@sheetalkamat
Merge branch 'master' into tscJsFiles
400b353
@sheetalkamat
Add option --jsExtensions to handle extensions to treat as javascript
c30104e 
- Command line now takes --jsExtension multiple times or comma separated list of extensions
- tsconfig accepts array of extension strings
RyanCavanaugh
RyanCavanaugh reviewed Oct 19, 2015
View reviewed changes
src/compiler/commandLineParser.ts
* @param s
* @param existingValue
*/
function parseMultiValueStringArray(s: string, existingValue: string[]) {
Copy link
Member

Choose a reason for hiding this comment

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

What's the difference between this and the built-in split ?

Copy link
Member Author

Choose a reason for hiding this comment

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

I could use split but then I would need to iterate over result again to make sure the strings are non empty and are correct.

@sheetalkamat sheetalkamat mentioned this pull request Oct 20, 2015
Closed
14 tasks
@sheetalkamat
Copy link
Member Author

This is now handled in #5345

@mhegazy mhegazy deleted the tscJsFiles branch November 2, 2017 21:02
@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
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sheetalkamat @RyanCavanaugh @mhegazy @vladima @msftclas