-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Airport: KCVG #272
Comments
@RNGFavorsNoobs if you want to open a PR, we can spin up a release app that will run your code. It'll update after every push. At least then you'll be able to check your work. |
@n8rzz , I'm nor exactly sure what a PR or a "release app" is, but if it means I can test it for myself, that would be awesome! |
yeah, open a pull request. Just thinking about it, I'm not sure if we can spin up a review app from a fork. But I'm curious, so lets give it a shot! |
If you head on over to the pull request, you will see a button towards the bottom |
... unless there is a failure. like there is now |
Was just about to say something, haha. |
looks like a JSON syntax error somewhere. try throwing the json you've created in here: http://jsonlint.com/ and then fix any lint errors, then re-commit. The app should rebuild on any new commits. |
Maybe the missing comma on line 3? |
There were two errors. Fixed and recommitting. Hold on. |
@RNGFavorsNoobs you can also join our slack channel http://slack.openscope.co/ (and you can be the first to test this little guy) |
Recommitted and joining the slack. |
@RNGFavorsNoobs we will close this once your PR is merged |
@n8rzz Whoops. : P |
Description in this pull request.
The text was updated successfully, but these errors were encountered: