Skip to content

Awesome typescript was complaining that backend is an unused property.#4

Merged
DarkXaHTeP merged 3 commits intoDarkXaHTeP:masterfrom
Squidex:master
Apr 9, 2017
Merged

Awesome typescript was complaining that backend is an unused property.#4
DarkXaHTeP merged 3 commits intoDarkXaHTeP:masterfrom
Squidex:master

Conversation

@SebastianStehle
Copy link
Copy Markdown
Contributor

No description provided.

@DarkXaHTeP
Copy link
Copy Markdown
Owner

DarkXaHTeP commented Apr 9, 2017

@SebastianStehle could you please tell about the reasons for this change? It doesn't make any sense to me at the moment and breaks the build due to changes in dependencies order (you need to update factory inside progress-http.module as well). Thanks!

@SebastianStehle
Copy link
Copy Markdown
Contributor Author

Sorry, changing the order was stupid. The reason for the change is that the typescript compiler is complaining about an unused property "backend".

@DarkXaHTeP
Copy link
Copy Markdown
Owner

@SebastianStehle Thanks for the explanation. Just update the order in module declaration as well and it should work

@DarkXaHTeP
Copy link
Copy Markdown
Owner

@SebastianStehle thanks for the fixes! I'll double check that everything is fine later today and let you know

@DarkXaHTeP DarkXaHTeP merged commit 479c440 into DarkXaHTeP:master Apr 9, 2017
@DarkXaHTeP
Copy link
Copy Markdown
Owner

@SebastianStehle published your changes to npm as 0.4.1

@SebastianStehle
Copy link
Copy Markdown
Contributor Author

Awesome, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants