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.

Sign up for GitHub

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

Jump to bottom

Add Node.js Continuous Integration Workflow #5

Merged
merged 2 commits into from
Mar 14, 2022
Merged

Add Node.js Continuous Integration Workflow #5

merged 2 commits into from
Mar 14, 2022

Conversation

R055A
Copy link
Contributor

@R055A R055A commented Mar 12, 2022

Solves issue #4

Add a Node.js continuous integration (CI) workflow that is automated using GitHub actions for:

  • clean install of dependencies
  • building the Backend project
  • running automated testing

CI workflow automation is run when triggered:

  • by changes pushed to any remote branch in any forked repo
  • by pull requests to the main branch in the main repository
  • manually using the GitHub website for the repository
  • on schedule every day at 5 am NZT (or 4 pm UTC)

Additional files:

  • package.json, including dependencies:
    • mocha
    • express
    • supertest
  • package-lock.json (for passing the CI workflow)
  • empty test file (for passing the CI workflow)
  • .gitignore, for ignoring:
    • node_modules/
    • .idea/

@R055A R055A added this to the Assignment 1 milestone Mar 12, 2022
@R055A R055A requested review from kimslor and hiin3d55 March 12, 2022 08:55
@R055A R055A self-assigned this Mar 12, 2022
@R055A R055A linked an issue Mar 12, 2022 that may be closed by this pull request
Add Node.js Continuous Integration workflow #4
Closed
@AlexMiao7 AlexMiao7 self-requested a review March 13, 2022 23:16
@R055A R055A requested a review from emettreio March 13, 2022 23:47
@R055A R055A requested a review from qzho236 March 13, 2022 23:48
@R055A R055A mentioned this pull request Mar 14, 2022
Closed
@R055A R055A added the enhancement New feature or request label Mar 14, 2022
hiin3d55
hiin3d55 approved these changes Mar 14, 2022
View reviewed changes
Copy link
Contributor

@hiin3d55 hiin3d55 left a comment

Choose a reason for hiding this comment

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

LGTM!

@R055A R055A merged commit 5b6d1d4 into SE701-T5:main Mar 14, 2022
@R055A R055A deleted the ci-workflow branch March 14, 2022 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexMiao7 AlexMiao7 AlexMiao7 approved these changes

@kimslor kimslor kimslor approved these changes

@hiin3d55 hiin3d55 hiin3d55 approved these changes

@emettreio emettreio Awaiting requested review from emettreio

@qzho236 qzho236 Awaiting requested review from qzho236

Assignees

@R055A R055A

Labels
enhancement New feature or request
Projects
None yet
Milestone
Assignment 1
Development

Successfully merging this pull request may close these issues.

Add Node.js Continuous Integration workflow
4 participants
@R055A @AlexMiao7 @kimslor @hiin3d55