Skip to content
This repository was archived by the owner on Sep 25, 2024. It is now read-only.

Bump golang version / set to latest golang alpine image#62

Open
NorskNoobing wants to merge 4 commits intojayden-chan:masterfrom
NorskNoobing:master
Open

Bump golang version / set to latest golang alpine image#62
NorskNoobing wants to merge 4 commits intojayden-chan:masterfrom
NorskNoobing:master

Conversation

@NorskNoobing
Copy link

@NorskNoobing NorskNoobing commented Sep 24, 2024

Description of the change

Sets the golang image in the DOCKERFILE to golang:alpine. This makes it so it always uses the latest golang alpine version, so the image building process won't break each time the golang package dependencies are updated.

Type of change

  • Refactoring (no change in functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation or Development tools (readme, specs, tests, code formatting)

Checklists

Development

  • Prettier was run on frontend
  • go fmt was run on backend
(if new config variable was added):
  • Configuration documentation was updated
  • Docker Compose example file was updated (for required variables with no default)
(if new dependency was added):
  • New dependencies have been audited and are justified
  • Development documentation has been updated
(if database schema was updated):
  • New database migrations have been generated with migrate tool
  • Migrations have been tested in both directions (cleanly migrates up and down)

Paperwork

  • This pull request has a descriptive title and information useful to a reviewer
  • AUTHORS.txt was updated if there are new contributors (welcome!)

Code review

  • Security impacts of this change have been considered

This update sets the image to the floating golang:alpine image. This makes it so it would always use the latest image, in order to keep up with dependencies and patch security vulnerabilities.
@NorskNoobing
Copy link
Author

This solves #60

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant