{"push_id":2701346588,"size":6,"distinct_size":1,"ref":"refs/heads/fmt","head":"31ee272b9e83a5bfd678e4b5380b78d9a08c04b2","before":"beed042fafb6cb75eecf336db97efa6650a76f77","commits":[{"sha":"66f6f5539f2d353667cd03fa79071dda3878a7bb","author":{"name":"Martin Probst","email":"54669547a225ff20cba8b75a4adca540eef25858@probst.io"},"message":"Fix emit for generic constructor functions.\n\nClosure does not support generic types in constructor functions (same as\nfor regular functions). Make sure to emit generic type arguments as `?`.","distinct":false,"url":"https://api.github.com/repos/angular/tsickle/commits/66f6f5539f2d353667cd03fa79071dda3878a7bb"},{"sha":"aeff8019c83bda43826efe2b540bdcbb99163fd9","author":{"name":"Martin Probst","email":"54669547a225ff20cba8b75a4adca540eef25858@probst.io"},"message":"Do not emit local type aliases.\n\nEmitting local type aliases is hard:\n- Introducing a new value symbol can conflict with existing values in\n  the same scope.\n- Creating a new name is hard as well, as type aliases might be\n  referenced before their declaration site, meaning tsickle would need\n  to do a two phase emit to produce correct names.\n- Introducing a value and then re-parsing can confuse TS on what symbols\n  are exported or not, causing duplicated exports.\n\nThis change side-steps the problem by consistently resolving type\naliases to their underlying type in all locations. Previously, we\nattempted to add code to avoid resolving aliases in certain locations.\nHowever this code never quite worked consistently (there's no guarantee\nin TS' API that alias names are retained), so there is functionality to\nwork around the problem in either case by emitting forward declares.","distinct":false,"url":"https://api.github.com/repos/angular/tsickle/commits/aeff8019c83bda43826efe2b540bdcbb99163fd9"},{"sha":"b561d8d914aa72f20b084db101dfbe694c83bdb6","author":{"name":"Martin Probst","email":"54669547a225ff20cba8b75a4adca540eef25858@probst.io"},"message":"Emit hard goog.require()s only for explicitly imported modules.\n\nPreviously, tsickle would emit goog.require() calls for any symbol that\nwas referenced in the current file if its module contained only types,\nregardless of whether the module was actually imported or not.\n\nThis can cause strict dependency checking errors if the hard required\nfile was not already a valid dependency of the current file.\n\nThis change avoids the problem by being careful not to emit such imports\nif the referenced module is not explicitly imported.","distinct":false,"url":"https://api.github.com/repos/angular/tsickle/commits/b561d8d914aa72f20b084db101dfbe694c83bdb6"},{"sha":"5b8278bc9b3b8762e31f4995a7750057be946331","author":{"name":"Evan Martin","email":"8e4c7829956390ca66ad4a9787c5043d687a289d@gmail.com"},"message":"clarify the release process\n\nDocument that you need to manually push the release change\nrather than merging it.","distinct":false,"url":"https://api.github.com/repos/angular/tsickle/commits/5b8278bc9b3b8762e31f4995a7750057be946331"},{"sha":"feec65817da0ac8c53f9546c13f1d49798038c51","author":{"name":"Evan Martin","email":"8e4c7829956390ca66ad4a9787c5043d687a289d@gmail.com"},"message":"drop unused entries from package.json\n\nThis was just done by eyeballing the list and thinking about\nwhich pieces we actually use.","distinct":false,"url":"https://api.github.com/repos/angular/tsickle/commits/feec65817da0ac8c53f9546c13f1d49798038c51"},{"sha":"31ee272b9e83a5bfd678e4b5380b78d9a08c04b2","author":{"name":"Evan Martin","email":"8e4c7829956390ca66ad4a9787c5043d687a289d@gmail.com"},"message":"drop gulp in favor of our own check_format script\n\nThe only reason we have gulp involved is for checking clang-format.\nI'd like to also check that README.md is formatted.  Rather than add\nmore gulp, I figured I'd go the other way and remove the gulp.\n\nPS: I wrote the checker as a JS program rather than a shell script to\nhave some hope of this working on Windows.","distinct":true,"url":"https://api.github.com/repos/angular/tsickle/commits/31ee272b9e83a5bfd678e4b5380b78d9a08c04b2"}]}
{"actor":{"display_login":"evmar"}}