Skip to content

Commit

Permalink
basic Appveyor configuration
Browse files Browse the repository at this point in the history
  • Loading branch information
Alexey Gurianov committed Jun 22, 2015
1 parent b918f1f commit 0f1c336
Showing 1 changed file with 21 additions and 0 deletions.
21 changes: 21 additions & 0 deletions appveyor.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
# Test against this version of Node.js
environment:
nodejs_version: "0.10"

# Install scripts. (runs after repo cloning)
install:
# Get the latest stable version of Node.js or io.js
- ps: Install-Product node $env:nodejs_version
# install modules
- npm install

# Post-install test scripts.
test_script:
# Output useful info for debugging.
- node --version
- npm --version
# run tests
- npm test

# Don't actually build.
build: off

6 comments on commit 0f1c336

@Guria
Copy link
Member

@Guria Guria commented on 0f1c336 Jun 22, 2015

Choose a reason for hiding this comment

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

@blond @sipayRT не пойму, почему под виндой билд упал. И тест и сама технология используют os.EOL. В логе AssertionError с идентичными строками.

@blond
Copy link
Member

@blond blond commented on 0f1c336 Jun 22, 2015

Choose a reason for hiding this comment

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

Технология генерит строки с EOL только внутри одного шаблона, а между шаблонами ставит \n из-за justJoinFiles.

А в тесте идёт проверка, что всё разделено с помощью EOL.

@Guria
Copy link
Member

@Guria Guria commented on 0f1c336 Jun 22, 2015

Choose a reason for hiding this comment

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

Спасибо. Какой вариант решения предложишь?

  • перейти везде на \n вместо EOL
  • отказаться от justJoinFiles
  • использовать </script>\n<script type="text/ng-template" id="block1.tmpl.html"> в тесте
  • забить на appveyor
  • дождаться релиз ENB с использованием EOL в build-flow

@blond
Copy link
Member

@blond blond commented on 0f1c336 Jun 22, 2015

Choose a reason for hiding this comment

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

Лучший вариант — кинуть PR в ENB ;)

@sipayRT
Copy link

Choose a reason for hiding this comment

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

Лучший вариант — кинуть PR в ENB ;)

отличный вариант :)

@Guria
Copy link
Member

@Guria Guria commented on 0f1c336 Jun 23, 2015

Choose a reason for hiding this comment

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

PR то уже кинут. Но для меня все ещё актуален мой 5ый вариант.

Please sign in to comment.