Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Updates for TypeScript 2.5 #651

Merged
merged 3 commits into from
Oct 13, 2017
Merged

Updates for TypeScript 2.5 #651

merged 3 commits into from
Oct 13, 2017

Conversation

kitsonk
Copy link
Member

@kitsonk kitsonk commented Aug 29, 2017

Type: feature

The following has been addressed in the PR:

  • There is a related issue
  • All code matches the style guide
  • Unit or Functional tests are included in the PR

Description:

This PR uses the RC for TypeScript 2.5. There are a couple minor changes where TypeScript has gotten a bit more strict on typing of some complex types.

References: dojo/meta#197

@dylans dylans added this to the 2017.08 milestone Aug 29, 2017
package.json Outdated
},
"dependencies": {
"@dojo/core": "next",
Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to add these as deps?

Copy link
Member Author

Choose a reason for hiding this comment

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

DOH! no... it was the freaking lack of --no-save and then me thinking it was just overwriting what was already there...

Copy link
Member

Choose a reason for hiding this comment

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

The next release of grunt-dojo2 should fix that 😄

@kitsonk kitsonk modified the milestones: 2017.08, 2017.09 Sep 4, 2017
@kitsonk kitsonk added the beta4 label Oct 9, 2017
@kitsonk kitsonk modified the milestones: 2017.09, 2017.10 Oct 9, 2017
@kitsonk
Copy link
Member Author

kitsonk commented Oct 13, 2017

Missed this was approved. Going to land it, and then see what 2.6 does.

@kitsonk kitsonk merged commit 75c112b into dojo:master Oct 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants