-
Notifications
You must be signed in to change notification settings - Fork 44
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
Site maintenance etc. #112
Conversation
✅ Deploy Preview for oci-conformance ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
65de304
to
df3fd11
Compare
Signed-off-by: Josh Dolitsky <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have Ruby installed locally, so this SGTM, but I'm far from an expert in this part of the code.
// See https://github.com/darccio/mergo#important-notes | ||
replace github.com/imdario/mergo => github.com/imdario/mergo v0.3.16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems like a sign we'd want to either update our code or swap out the dependency.
# build-static-site.sh | ||
# Generates a static HTML site into _site/ | ||
# Requires Go ...and Ruby |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be useful to make this available to run with a docker container (e.g. with a make serve
command that does a docker build / docker run
) for developers to test locally.
@sudo-bmitch - I converted your two comments into issues:
Will go ahead and merge this since it gets us into a better spot |
./hack/build-static-site.sh