Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Fixes for parity-extension #6990

Merged
merged 5 commits into from
Nov 9, 2017
Merged

Fixes for parity-extension #6990

merged 5 commits into from
Nov 9, 2017

Conversation

tomusdrw
Copy link
Collaborator

@tomusdrw tomusdrw commented Nov 6, 2017

No description provided.

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Nov 6, 2017
@tomusdrw tomusdrw requested a review from jacogr November 6, 2017 09:43
@5chdn 5chdn added this to the 1.9 milestone Nov 6, 2017
@@ -21,7 +21,7 @@
"Parity"
],
"scripts": {
"build": "npm run build:lib && npm run build:app",
"build": "npm run build:lib && npm run build:app && npm run build:embed",
Copy link
Contributor

Choose a reason for hiding this comment

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

The only issue here is line 27 - fails on Windows.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Solved by using cross-env

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Nov 9, 2017
@jacogr jacogr merged commit aaeb5d4 into master Nov 9, 2017
@jacogr jacogr deleted the td-extension branch November 9, 2017 12:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants