Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Add Flow step to pipeline.yml #37

Merged
merged 3 commits into from
Dec 5, 2017
Merged

Add Flow step to pipeline.yml #37

merged 3 commits into from
Dec 5, 2017

Conversation

AlexMSmithCA
Copy link
Member

Created with fusion-orchestrate.

@AlexMSmithCA
Copy link
Member Author

Update:

  • Some minor Flow annotation changes to fix check complaints.
  • Using comment-syntax here to avoid breaking tests - which don't appear to transform away Flow syntax

Copy link
Contributor

@KevinGrandon KevinGrandon left a comment

Choose a reason for hiding this comment

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

Looks good, but not sure if we want to use comment syntax? What unit tests were failing, shouldn't they be run through babel?

@rtsao
Copy link
Member

rtsao commented Dec 5, 2017

!merge

@old-fusion-bot old-fusion-bot bot merged commit 1ed5fdf into fusionjs:master Dec 5, 2017
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.

3 participants