Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

Add StandardJS (closes #323) #328

Merged
merged 2 commits into from Oct 27, 2017
Merged

Add StandardJS (closes #323) #328

merged 2 commits into from Oct 27, 2017

Conversation

j-a-m-l
Copy link
Contributor

@j-a-m-l j-a-m-l commented Oct 9, 2017

This PR adds StandardJS (#323) with the configuration for this project.

I've adapted the code under the test folder to that code style too. The rest of the code should be done incrementally because it is composed of a lot of different parts.

@j-a-m-l j-a-m-l changed the title Add StandardJS Add StandardJS (closes #323) Oct 9, 2017
return this.click('[aria-label="Switch Network"] button')
.pause(500)
.click("//button[contains(text(), '"+network+"')]")
.click("//button[contains(text(), '" + network + "')]")

Choose a reason for hiding this comment

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

Is there an option in standardJS to enforce template literals?

@j-a-m-l
Copy link
Contributor Author

j-a-m-l commented Oct 12, 2017

@spresnal In that case I kept it that way because I'm waiting to having more parts refactored (#298) before changing to a more recent Node.js version (#299)

$scope = _$rootScope_.$new()
})
})
// beforeEach( () => {
Copy link

Choose a reason for hiding this comment

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

should be removed not commented out

@luciorubeens luciorubeens merged commit 1197863 into ArkEcosystem:master Oct 27, 2017
luciorubeens added a commit that referenced this pull request Oct 27, 2017
@luciorubeens
Copy link
Contributor

Merged manually! +5 👍🏻

bag-man added a commit to bag-man/ark-rpc that referenced this pull request Oct 27, 2017
I saw that stardardJS was used here:
ArkEcosystem/desktop-wallet#328 so figure it would
be good to match that.

This is currently not working due to the network connect middleware
being ran in the wrong context.
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.

None yet

4 participants