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

Fix output selection for the AST #38

Merged
merged 2 commits into from
Dec 19, 2017
Merged

Fix output selection for the AST #38

merged 2 commits into from
Dec 19, 2017

Conversation

axic
Copy link
Contributor

@axic axic commented Nov 29, 2017

Fixes #37.

Copy link
Contributor

@gnidan gnidan left a comment

Choose a reason for hiding this comment

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

@axic thanks for this - question for you before I merge

@@ -76,9 +76,11 @@ var compile = function(sources, options, callback) {
optimizer: options.solc.optimizer,
outputSelection: {
"*": {
"": [
"legacyAST"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the rename?

Copy link
Contributor

@gnidan gnidan left a comment

Choose a reason for hiding this comment

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

Oh nevermind actually - I realized legacyAST is what was being used

@gnidan gnidan merged commit 6ed07aa into trufflesuite:develop Dec 19, 2017
@axic axic deleted the solc-ast branch December 19, 2017 21:45
@gnidan gnidan mentioned this pull request Jan 24, 2018
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.

2 participants