Skip to content

Conversation

@LePaVuki
Copy link
Owner

@LePaVuki LePaVuki commented Oct 19, 2024

DevOps task formal PR for the code review

The project consists of:

  • Base repository with Angular and Node
  • Dockerfile to set up an image of Nginx
  • Nginx configuration file to serve an Angular app
  • GitHub workflow, which runs following job on every push/PR to master:
    • Installs/Tests/Builds the Angular app
    • Builds a docker image based on the Angular app served by Nginx
    • Pushes created image to the GitHub registry

LePaVuki and others added 30 commits October 17, 2024 03:42
Initial test of GitHub actions
changed ti install, test
trying with cd command
attempt with npm ci and actions/setup-node@v3
attempt with "working-directory" parameters
running:
- npm run start
- npm run test
fixed git fail
Changed dockerfile to give nginx user privileges and deleted user mode from nginx.conf
Copy link
Collaborator

@kseniyashaydurova kseniyashaydurova left a comment

Choose a reason for hiding this comment

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

Needs a bit more changes

@kseniyashaydurova kseniyashaydurova requested review from kseniyashaydurova and removed request for kseniyashaydurova November 6, 2024 05:28
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.

4 participants