Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@timothywarner
Copy link
Owner

Combined fixes from PRs #17 and #18

Pipeline Updates

  • Fixed scripts to match nodeapp-1 actual npm commands
  • Updated test result publishing for mochawesome HTML reports
  • Added workingDirectory variables
  • Removed non-existent build/coverage scripts

nodeapp-1 Fixes

  • Added missing config files (.eslintrc.json, .mocharc.json)
  • Fixed app.js to export properly for testing
  • Updated all tests to use app instance
  • Added Dockerfile and production setup
  • Created comprehensive README

New Files

  • nodeapp-1/.eslintrc.json - ESLint configuration
  • nodeapp-1/.mocharc.json - Mocha test configuration
  • nodeapp-1/Dockerfile - Container support
  • nodeapp-1/.dockerignore - Docker ignore rules
  • nodeapp-1/server.js - Production entry point
  • pipelines/single-stage-ci.yml - Basic CI pipeline
  • pipelines/multi-stage-cicd.yml - Full CI/CD pipeline

All conflicts resolved. Ready to merge.

@timothywarner timothywarner merged commit 2354c94 into master Jul 19, 2025
0 of 3 checks passed
@timothywarner timothywarner deleted the combined-fixes branch July 19, 2025 23:06
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