Skip to content

Strip leading empty lines in flow generated code#1127

Merged
trevor-scheer merged 4 commits intoapollographql:masterfrom
ganemone:master
Mar 19, 2019
Merged

Strip leading empty lines in flow generated code#1127
trevor-scheer merged 4 commits intoapollographql:masterfrom
ganemone:master

Conversation

@ganemone
Copy link
Copy Markdown
Contributor

@ganemone ganemone commented Mar 18, 2019

This updates apollo-codegen-flow to strip leading empty lines in flow generated code. This is required to ensure the generated code passes lint, since eslint will only be disabled for all the code after the /* eslint-disable */ comment.

  • Update CHANGELOG.md* with your change (include reference to issue & this PR)
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass

Copy link
Copy Markdown
Contributor

@trevor-scheer trevor-scheer left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @ganemone! 🎉

@trevor-scheer trevor-scheer merged commit e878c9c into apollographql:master Mar 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants