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

Build and use our image in CI. #738

Closed
wants to merge 1 commit into from
Closed

Conversation

bfabio
Copy link
Member

@bfabio bfabio commented Oct 2, 2020

Waiting for Hacktoberfest end to refine and merge, in order to avoid breaking something for the newcomers.

Fix #691.

@bfabio bfabio force-pushed the circleci branch 6 times, most recently from 8bec349 to 2bc195d Compare October 11, 2020 12:56
Build the Docker image and use it in CI to run tests
and build the site. Also upload it to DockerHub.
@@ -25,7 +25,7 @@
"scripts": {
"build": "webpack --config webpack.prod.js",
"lint": "npx eslint _site/ --ext .json",
"deploy": "gh-pages -u 'Deploy Bot <[email protected]>' -d _site"
"deploy": "npx gh-pages -u 'Deploy Bot <[email protected]>' -d _site"
Copy link
Member

@sebbalex sebbalex Oct 12, 2020

Choose a reason for hiding this comment

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

+1 we can even remove it from dependencies

@bfabio
Copy link
Member Author

bfabio commented Apr 7, 2022

This wouldn't apply anymore after #1178, besides, the need for this is just a symptom of the two different environments (webpack + jekyll) and sort of convoluted build. We simplified a great deal of that, but still not enough IMO.

#1007 should be the ideal solution.

Closing this.

@bfabio bfabio closed this Apr 7, 2022
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.

Build the Docker image as part of the CI pipeline and use it
2 participants