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

Upgraded Jekyll version utilized in Dockerfile #41

Merged
merged 2 commits into from
Jul 17, 2023

Conversation

dpeachpeach
Copy link
Contributor

Successfully served data-and-lab locally. I looked for opportunities to upgrade Jekyll where I could and only saw an improvement I could make in the Dockerfile.
Something I have a question about is the way data-and-lab runs without utilizing a Gemfile (All the docs I found online stated that a gemfile was essential and in the practice jekyll website I made I utilized a gemfile).
Kepler.Gl integration next.

@dpeachpeach
Copy link
Contributor Author

I didn't merge using an issue, that was my bad, going to do so for Kepler.gl integration.

@@ -1,7 +1,7 @@
# Pull down a Jekyll base image (which uses ruby:2.7.1-alpine3.11)

# dev-stage
FROM jekyll/jekyll:4.0
FROM jekyll/jekyll:4.2.2
Copy link
Member

Choose a reason for hiding this comment

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

@dpeachpeach Did you try it on Apple M1? It seems not working correctly on arm64. Thanks!

@lixun910
Copy link
Member

@dpeachpeach I can't get local dev working with docker. Instead, I just upgraded gulp to v4, so with node v12, one can run local dev using gulp. You need to rerun npm install and then gulp. Another simple way to do: you can just run jekyll build and jekyll serve to get local dev working.

Let me know what you think. Thanks!

"url": "https://github.com/GeoDaCenter/data-and-lab.git"
},
"volta": {
"node": "12.22.12"
Copy link
Member

Choose a reason for hiding this comment

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

You can use volta or nvm to pin a specific node version

Copy link
Member

@lixun910 lixun910 left a comment

Choose a reason for hiding this comment

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

You can merge it once you verify it on your side. Thanks, @dpeachpeach !

@dpeachpeach
Copy link
Contributor Author

Thanks Xun! I'm going to verify it right now

@lixun910
Copy link
Member

@dpeachpeach If everything works as expected, you can click the green "Squash and merge" button. Thanks!

@dpeachpeach dpeachpeach merged commit 40ee839 into GeoDaCenter:gh-pages Jul 17, 2023
@lixun910 lixun910 mentioned this pull request Jul 19, 2023
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