Skip to content

Changes to typeWriter and type baselines#554

Merged
JsonFreeman merged 13 commits into
masterfrom
typeBaselines
Aug 29, 2014
Merged

Changes to typeWriter and type baselines#554
JsonFreeman merged 13 commits into
masterfrom
typeBaselines

Conversation

@JsonFreeman
Copy link
Copy Markdown
Contributor

This PR is not to be merged in wholesale. I am just documenting the changes I've made in this branch to compare the type baselines with the old compiler's type baselines. The changes were in the service of making the new baselines and old baselines look as similar as possible. The idea is to pick and choose the changes we actually want to port to master. Various smaller / more nuanced differences were captured in the form of bugs with the prefix [typeBaselines]. The majority of the changes ported should be source changes, but a few test changes may be ported as well.

@JsonFreeman
Copy link
Copy Markdown
Contributor Author

The Travis failure is a known issue #555

Comment thread src/compiler/checker.ts Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i would remove this, i do not think it makes sense to specialize this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Logging a bug for it (#559) and reverting back to original behavior.

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Aug 27, 2014

👍

Comment thread src/harness/typeWriter.ts Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

probably do not need useTypeOfFunction flag as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

👍

@JsonFreeman
Copy link
Copy Markdown
Contributor Author

I am not sure why the Travis build is failing. I am unable to reproduce the 2 test failures locally.

@JsonFreeman
Copy link
Copy Markdown
Contributor Author

Travis seems to be running the tests from master on the typeBaselines branch.

JsonFreeman added a commit that referenced this pull request Aug 29, 2014
Changes to typeWriter and type baselines
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants