Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

CI: Configure AppVeyor #937

Closed
wants to merge 3 commits into from
Closed

CI: Configure AppVeyor #937

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
24a1883
CI: Adds appveyor.yml.
am11 Jan 20, 2015
e3b3c98
Package: Updates dependencies.
am11 Jan 20, 2015
6811bd9
README: Adds appveyor badge.
am11 Jan 20, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
[![Build Status](https://travis-ci.org/jscs-dev/node-jscs.svg?branch=master)](https://travis-ci.org/jscs-dev/node-jscs)
[![Windows CI](https://ci.appveyor.com/api/projects/status/github/jscs-dev/node-jscs?svg=true)](https://ci.appveyor.com/project/jscs-dev/node-jscs/branch/master)
[![Coverage Status](https://img.shields.io/coveralls/jscs-dev/node-jscs.svg?style=flat)](https://coveralls.io/r/jscs-dev/node-jscs?branch=master)
[![Dependency Status](https://david-dm.org/jscs-dev/node-jscs.svg?theme=shields.io&style=flat)](https://david-dm.org/jscs-dev/node-jscs)
[![devDependency Status](https://david-dm.org/jscs-dev/node-jscs/dev-status.svg?theme=shields.io&style=flat)](https://david-dm.org/jscs-dev/node-jscs#info=devDependencies)
Expand Down Expand Up @@ -31,3 +32,4 @@ including presets from popular style guides like jQuery, Airbnb, Google, and mor
* [Moment.js](http://momentjs.com/)
* [TodoMVC](http://todomvc.com/)
* [Famous](http://famo.us/)

12 changes: 12 additions & 0 deletions appveyor.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
version: "{build}"

install:
- ps: Install-Product node $env:nodejs_version
- node --version
- npm --version
- npm install
Copy link
Member

Choose a reason for hiding this comment

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

I think this line should be moved to install section (because it's about installation ;-)


test_script:
- npm test

build: off
Copy link
Member

Choose a reason for hiding this comment

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

Can you give me link to this setting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure: http://www.appveyor.com/docs/appveyor-yml (Ctrl+F, search for: '174')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They use build slug for msbuild systems (mostly Visual Studio projects).

28 changes: 14 additions & 14 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,32 +63,32 @@
"cli-table": "~0.3.1",
"colors": "~1.0.3",
"commander": "~2.6.0",
"esprima": "~1.2.4",
"esprima": "~2.0.0",
"esprima-harmony-jscs": "1.1.0-tolerate-import",
"estraverse": "~1.9.0",
"estraverse": "~1.9.1",
"exit": "~0.1.2",
"glob": "~4.3.5",
"lodash.assign": "~3.0.0",
"minimatch": "~2.0.0",
"minimatch": "~2.0.1",
"prompt": "~0.2.14",
"strip-json-comments": "~1.0.1",
"strip-json-comments": "~1.0.2",
"supports-color": "~1.2.0",
"vow": "~0.4.3",
"vow-fs": "~0.3.1",
"xmlbuilder": "~2.4.0"
"vow": "~0.4.8",
"vow-fs": "~0.3.4",
"xmlbuilder": "~2.5.0"
},
"devDependencies": {
"browserify": "~8.1.1",
"coveralls": "~2.11.1",
"has-ansi": "~1.0.0",
"browserify": "~8.1.3",
"coveralls": "~2.11.2",
"has-ansi": "~1.0.1",
"jshint": "~2.6.0",
"mocha": "~2.1.0",
"regenerate": "~1.2.1",
"rewire": "~2.1.0",
"sinon": "~1.12.0",
"rewire": "~2.1.5",
"sinon": "~1.12.2",
Copy link
Member

Choose a reason for hiding this comment

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

Is dependencies update required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nops, just did it in a separate commit. All is working fine though.

Copy link
Member

Choose a reason for hiding this comment

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

Alrighty, it's fine, could merge it with it

"unicode-7.0.0": "~0.1.5",
"unit-coverage": "~3.3.0",
"xml2js": "~0.4.2"
"xml2js": "~0.4.4"
},
"bin": {
"jscs": "./bin/jscs"
Expand All @@ -113,7 +113,7 @@
},
"scripts": {
"lint": "jshint . && node bin/jscs lib test bin publish",
"test": "npm run lint && mocha",
"test": "npm run lint && mocha --color",
"coverage": "unit-coverage run -p common",
"coverage-html": "unit-coverage run -p common -r html -o coverage.html",
"browserify": "browserify --standalone JscsStringChecker lib/string-checker.js -o jscs-browser.js",
Expand Down