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

Reworking Dockerfile, CI, NPM Scripts #222

Draft
wants to merge 6 commits into
base: unstable
Choose a base branch
from

Conversation

Kawanaao
Copy link
Contributor

@Kawanaao Kawanaao commented Mar 30, 2024

In this pull request I want to change the scripts for development, debugging and publishing, for now the goal is

  • Minimize the size of the output container while maintaining functionality
  • Caching
  • Rewriting Circle CI for new Dockerfiles
  • Adding generalized scripts to NPM, including content generators (Halfway with generators)
  • Adding Github Workflows files or at least integration with Circle CI, for PR validation

@cla-bot cla-bot bot added the cla-signed label Mar 30, 2024
@Kawanaao
Copy link
Contributor Author

I think maybe it’s worth transferring the meta data according to the principles of OpenContainers Annotations? Instead of meta.json

@BearTS
Copy link
Contributor

BearTS commented Jun 7, 2024

Hey, it may be worthy to utilise https://github.com/air-verse/air for running dev version of the server by default.
I can see myself making multiple changes and then having to restart the build and restart server right now
Let me know about your thoughts on this
@Kawanaao

@BearTS
Copy link
Contributor

BearTS commented Jun 7, 2024

I can see that this has not been updated since the last 2 months.
I can take over if @Kawanaao or @azukaar agrees.

@azukaar
Copy link
Owner

azukaar commented Jun 11, 2024

Whatever you do make sure you make small incremental PRs :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants