Skip to content

bump to v0.0.9 of cc-yaml#59

Merged
ABaldwinHunter merged 1 commit intomasterfrom
abh_bump_cc_yaml_to_v0.0.9
Jul 20, 2015
Merged

bump to v0.0.9 of cc-yaml#59
ABaldwinHunter merged 1 commit intomasterfrom
abh_bump_cc_yaml_to_v0.0.9

Conversation

@ABaldwinHunter
Copy link
Copy Markdown

@ABaldwinHunter ABaldwinHunter commented Jul 17, 2015

@codeclimate/review bump cc-yaml from v0.0.7 to v0.0.9, which fixes bug in v0.0.8 and keeps v0.0.8 new functionality:

  1. Adds warning if no engines or languages key is found.
$ codeclimate validate-config
WARNING: No languages or engines key found. Must have analysis key.
  1. Updates language in warning for when both engines and languages keys are present.

old warning:

$ codeclimate validate-config
WARNING: engines key already found, dropping key: languages. Analysis settings for Languages and 
Engines are both valid but mutually exclusive. Note: command line analysis requires an Engines 
configuration.

new warning:

$ codeclimate validate-config
WARNING: Use either a Languages key or an Engines key, but not both. They are mutually exclusive.

@gdiggs
Copy link
Copy Markdown
Contributor

gdiggs commented Jul 20, 2015

LGTM

ABaldwinHunter pushed a commit that referenced this pull request Jul 20, 2015
Merge pull request #59 from codeclimate/abh_bump_cc_yaml_to_v0.0.9
7c8235f 
bump to v0.0.9 of cc-yaml
@ABaldwinHunter ABaldwinHunter merged commit 7c8235f into master Jul 20, 2015
@ABaldwinHunter ABaldwinHunter deleted the abh_bump_cc_yaml_to_v0.0.9 branch July 20, 2015 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ABaldwinHunter @gdiggs