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.

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

Bump dependencies + devcontainer + Makefile + node-service improvement #30

Merged
merged 8 commits into from
May 10, 2024

Conversation

mathieu-benoit
Copy link
Contributor

@mathieu-benoit mathieu-benoit commented May 10, 2024

  • Fix Node.js 16 actions are deprecated
  • Optimize node-service's container image and bump to alpine:3.19.1
  • Fix CVEs: mysql2 3.9.7 for node-service
  • Add devcontainer for node-service and podinfo, great way to demo VS Code lint, score-compose, humctl already pre-installed
  • Makefile / score-compose for node-service and podinfo
  • node-service - add MESSAGE and MySQL version/host
  • Update App, versus Workload, versus Container names to avoid confusion

@mathieu-benoit mathieu-benoit requested a review from a team as a code owner May 10, 2024 12:29
@mathieu-benoit mathieu-benoit marked this pull request as draft May 10, 2024 12:29
@mathieu-benoit mathieu-benoit changed the title Bump dependencies Bump dependencies + devcontainer May 10, 2024
@mathieu-benoit mathieu-benoit changed the title Bump dependencies + devcontainer Bump dependencies + devcontainer + Makefile + node-service improvement May 10, 2024
@mathieu-benoit mathieu-benoit marked this pull request as ready for review May 10, 2024 21:01
@mathieu-benoit
Copy link
Contributor Author

Tested in a test instance, working great!

Thanks for your review!

@mathieu-benoit mathieu-benoit merged commit 88178c8 into main May 10, 2024
2 checks passed
@mathieu-benoit mathieu-benoit deleted the bump branch May 10, 2024 22:59
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