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

feat: Adds appveyor build file #55

Merged
merged 5 commits into from
Jun 8, 2017
Merged

feat: Adds appveyor build file #55

merged 5 commits into from
Jun 8, 2017

Conversation

joshwiens
Copy link
Member

@joshwiens joshwiens commented May 22, 2017

Adds an appveyor build file to webpack-defaults

Output ...

branches:
  only:
    - master

init:
  - git config --global core.autocrlf input

environment:
  matrix:
    - nodejs_version: 7
      webpack_version: 2.2.0
      job_part: test
    - nodejs_version: 6
      webpack_version: 2.2.0
      job_part: test
    - nodejs_version: 4.3
      webpack_version: 2.2.0
      job_part: test

install:
  - ps: Install-Product node $env:nodejs_version x64
  - npm install

build: off

matrix:
  fast_finish: true

before_test:
  - cmd: npm install webpack@^%webpack_version%

test_script:
  - node --version
  - npm --version
  - cmd: npm run appveyor:%job_part%

Closes #54

@sapegin
Copy link
Member

sapegin commented Jun 6, 2017

Do we need someone else’s review or I can merge it?

@bebraw
Copy link
Contributor

bebraw commented Jun 6, 2017

@d3viant0ne Do you have any changes in mind for this? I don't understand Appveyor well enough to comment.

@joshwiens
Copy link
Member Author

I wanted to try this config out in a project before I merged it, to make sure it works the way I think it should.

@joshwiens joshwiens force-pushed the d3viant0ne-Appveyor branch from 001bc2e to 12e6eb2 Compare June 7, 2017 21:15
environment: {
matrix: [
{
nodejs_version: '7',
Copy link
Member

Choose a reason for hiding this comment

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

7 => 8 :)

Copy link
Member Author

Choose a reason for hiding this comment

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

@michael-ciniawsky - Now configured to use the config for it's nodejs versions like the travis template.

Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

👍 :shipit:

@joshwiens joshwiens merged commit b0f5110 into master Jun 8, 2017
@joshwiens joshwiens deleted the d3viant0ne-Appveyor branch June 8, 2017 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants