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

Export commands and scripts #177

Merged
merged 2 commits into from
Oct 4, 2018
Merged

Export commands and scripts #177

merged 2 commits into from
Oct 4, 2018

Conversation

facuspagnuolo
Copy link
Contributor

Fixes #40

I exported both scripts and commands

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.

Looks great! Should we also consider exporting runWithTruffle or similar as well, or these should work out of the box when run via a truffle exec?

import status from './status'
import unlink from './unlink'
import update from './update'
import verify from './verify'
Copy link
Contributor

Choose a reason for hiding this comment

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

Dat OCD


export {
version,
local,
network,
Copy link
Contributor

Choose a reason for hiding this comment

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

I like the idea of exporting the Controllers as well!

@spalladino spalladino added the status:ready-to-merge Order mergify to merge label Oct 4, 2018
@facuspagnuolo
Copy link
Contributor Author

Let's add runWithTruffle, agree :)

@mergify mergify bot merged commit 1e378d7 into master Oct 4, 2018
@facuspagnuolo facuspagnuolo deleted the export_scripts_and_commands branch October 22, 2018 18: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