Skip to content

Create npm-grunt.yml#14

Open
PearlsAmos wants to merge 1 commit intoOpenBuilders:mainfrom
PearlsAmos:main
Open

Create npm-grunt.yml#14
PearlsAmos wants to merge 1 commit intoOpenBuilders:mainfrom
PearlsAmos:main

Conversation

@PearlsAmos
Copy link

First card 21082025

First card 21082025
@navedjuwale-creator
Copy link

Thanks for adding the npm-grunt.yml workflow! This is a great first step toward automating your build process.

A few suggestions to ensure it runs smoothly:

· Confirm that the package.json includes the necessary Grunt scripts (e.g., grunt test) and that Grunt is listed as a dev dependency.
· Consider adding a pull_request trigger so the workflow runs on PRs, not just pushes.
· Specify a Node version (e.g., node-version: 16) to guarantee consistency.
· If you’re using any secrets (like npm tokens), make sure they’re configured in the repository secrets.

Let me know if you need help tweaking the file—happy to review again once updates are pushed!

Copy link

@navedjuwale-creator navedjuwale-creator left a comment

Choose a reason for hiding this comment

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

name: NodeJS with Grunt

on:
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]

jobs:
build:
runs-on: ubuntu-latest

strategy:
  matrix:
    node-version: [18.x, 20.x]

steps:
- uses: actions/checkout@v4

- name: Use Node.js ${{ matrix.node-version }}
  uses: actions/setup-node@v4
  with:
    node-version: ${{ matrix.node-version }}
    cache: 'npm'

- run: npm ci
- run: npx grunt test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants