Skip to content
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

Linting, coverage, coveralls and dev rocks #39

Merged
merged 1 commit into from
Feb 27, 2015
Merged

Conversation

thibaultcha
Copy link
Member

This implements:

  • A linter: make lint
  • Coverage for unit tests (only unit tested modules)
  • Coveralls integration (with README badge)
  • A dev_rocks.sh script for required rocks in development (ran by make dev, only installing dev rocks if they are missing)
  • A fix for kong.constants inconsistencies

@thibaultcha thibaultcha force-pushed the feature/coverage branch 2 times, most recently from 7e3a63f to ae2adcb Compare February 27, 2015 02:30
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 496c1e6 on feature/coverage into * on master*.

@thibaultcha thibaultcha force-pushed the feature/coverage branch 3 times, most recently from 99f5433 to 496c1e6 Compare February 27, 2015 05:31
@thibaultcha thibaultcha self-assigned this Feb 27, 2015
thibaultcha added a commit that referenced this pull request Feb 27, 2015
Linting, coverage, coveralls and dev rocks
@thibaultcha thibaultcha merged commit 7e138d3 into master Feb 27, 2015
@thibaultcha thibaultcha deleted the feature/coverage branch February 27, 2015 19:51
ctranxuan pushed a commit to streamdataio/kong that referenced this pull request Aug 25, 2015
Linting, coverage, coveralls and dev rocks
gszr pushed a commit that referenced this pull request Jun 17, 2021
… AWS API Gateway compatibility (#39)

honor the skip_large_bodies config setting even when not using aws api gateway compatibility. requires checking for bodies that exceeded the max client buffer size and reading those payloads from disk
hutchic pushed a commit that referenced this pull request Jun 10, 2022
fix(copyright) include the Kong COPYRIGHT
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