Skip to content

Commit

Permalink
Transition to Github Actions (Modernizr#2658)
Browse files Browse the repository at this point in the history
* Delete .travis.yml

* Try again to create Github Actions

* Remove global? Seems to cause problems

* Try to add Windows support

* Remove AppVeyor as GA replaces it :(

* Change README

* Remove specific branch (to help the renaming)

* Update node versions

* Remove appveyor from legacy grunt (to see if github doesnt complain)

* Update README

* Update dependencies

* Update testing actions

Co-authored-by: Markel F <git@markel.dev>
  • Loading branch information
rejas and Markel authored Jul 10, 2021
1 parent 689165a commit 70d1079
Show file tree
Hide file tree
Showing 7 changed files with 56 additions and 78 deletions.
26 changes: 26 additions & 0 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
name: Testing

on: [push, pull_request]

jobs:
build:
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
node-version: [12.x, 14.x, 16.x]
steps:
- name: Checkout code
uses: actions/checkout@v2
- name: Installing Node ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
- name: Installing dependencies
run: npm ci
- name: Installing gulp command line interface
run: npm install gulp-cli
- name: Testing Modernizr
run: npm test
- name: Sending Coverage
run: ./node_modules/.bin/nyc report --reporter=text-lcov | ./node_modules/.bin/codecov --pipe
21 changes: 0 additions & 21 deletions .travis.yml

This file was deleted.

5 changes: 2 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# Modernizr
[![npm version](https://badge.fury.io/js/modernizr.svg)](https://badge.fury.io/js/modernizr)
[![Build Status](https://api.travis-ci.org/Modernizr/Modernizr.svg?branch=master)](https://travis-ci.org/Modernizr/Modernizr)
[![Build Status](https://ci.appveyor.com/api/projects/status/github/Modernizr/modernizr?branch=master&svg=true)](https://ci.appveyor.com/project/rejas/modernizr)
[![Build Status](https://github.com/Modernizr/Modernizr/workflows/Testing/badge.svg)](https://github.com/Modernizr/Modernizr/actions)
[![codecov](https://codecov.io/gh/Modernizr/Modernizr/branch/master/graph/badge.svg)](https://codecov.io/gh/Modernizr/Modernizr)
[![Inline docs](https://inch-ci.org/github/Modernizr/Modernizr.svg?branch=master)](https://inch-ci.org/github/Modernizr/Modernizr)

Expand All @@ -19,7 +18,7 @@ Modernizr tests which native CSS3 and HTML5 features are available in the curren

## Breaking changes with v4

- Dropped Node 8 Support, please upgrade to Node v10
- Dropped Node 10 Support, please upgrade to Node v12

- Following tests got renamed:

Expand Down
24 changes: 0 additions & 24 deletions appveyor.yml

This file was deleted.

4 changes: 1 addition & 3 deletions legacy/Gruntfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -248,9 +248,7 @@ module.exports = function(grunt) {
*/
var tests = ['clean', 'eslint', 'pug', 'instrument', 'env:coverage', 'nodeTests'];

if (process.env.APPVEYOR) {
grunt.registerTask('test', tests);
} else if (process.env.BROWSER_COVERAGE !== 'true') {
if (process.env.BROWSER_COVERAGE !== 'true') {
grunt.registerTask('test', tests.concat(['generate', 'browserTests']));
} else {
grunt.registerTask('test', tests.concat(['generate', 'storeCoverage', 'browserTests', 'saucelabs-custom', 'makeReport', 'coveralls']));
Expand Down
52 changes: 26 additions & 26 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@
"codecov": "^3.8.2",
"del": "^6.0.0",
"eslint": "^7.30.0",
"eslint-plugin-jsdoc": "^35.4.2",
"eslint-plugin-jsdoc": "^35.4.3",
"find-parent-dir": "^0.3.1",
"fs-extra": "^10.0.0",
"globby": "^11.0.4",
Expand Down

0 comments on commit 70d1079

Please sign in to comment.