Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Detect initial values in fields declarations #241

Merged
merged 3 commits into from
Oct 12, 2018

Conversation

jbcarpanelli
Copy link
Contributor

Fixes #80.

@jbcarpanelli jbcarpanelli added the status:to-review Awaiting review label Oct 12, 2018
@jbcarpanelli jbcarpanelli force-pushed the detect-initial-values-in-fields-declarations branch from 0a70ac8 to 89f0a53 Compare October 12, 2018 07:59
@spalladino spalladino self-assigned this Oct 12, 2018
Copy link
Contributor

@spalladino spalladino left a comment

Choose a reason for hiding this comment

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

Awesome work Jota!!

}

function detectInitialValues(contractClass) {
const nodes = contractClass.ast.nodes.filter((n) => n.name === contractClass.contractName)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can be a find instead of filter, as there should be only one.

@spalladino spalladino added status:ready-to-merge Order mergify to merge and removed status:to-review Awaiting review labels Oct 12, 2018
@mergify mergify bot merged commit fea5ffa into master Oct 12, 2018
@jbcarpanelli jbcarpanelli deleted the detect-initial-values-in-fields-declarations branch October 17, 2018 19:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:ready-to-merge Order mergify to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants