-
Notifications
You must be signed in to change notification settings - Fork 1
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
(WIP) feat: Apply Next.js Blog Theme #6
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Charlie <[email protected]>
Refactor code around dark/light theme
Add build command in .toml
* adds lint command and eslintrc.json * adds prettier * makes formatting changes from prettier Co-authored-by: Tara Z Manicsic <[email protected]>
* updates text in new post section * adds the video * fixes faulty spacing
* starts update to env var section * --amend * add video and and link to walkthrough
❌ Deploy Preview for ble-testsuite failed.
|
From https://app.netlify.com/sites/ble-testsuite/deploys/645f8263149a8300088739ba Deploy summary
|
Deploy log from https://app.netlify.com/sites/ble-testsuite/deploys/645f8263149a8300088739ba
|
With reference to the error in the Netlify Deploy log
As a matter of fact, gmaca@alpha MINGW64 /e/data/gmaca/github/gmacario/ble-testsuite (feat/nextjs-blog)
$ npm i -D cypress
npm WARN idealTree Removing dependencies.cypress in favor of devDependencies.cypress
changed 3 packages, and audited 1045 packages in 2m
252 packages are looking for funding
run `npm fund` for details
16 vulnerabilities (8 moderate, 8 high)
To address issues that do not require attention, run:
npm audit fix
To address all issues (including breaking changes), run:
npm audit fix --force
Run `npm audit` for details.
gmaca@alpha MINGW64 /e/data/gmaca/github/gmacario/ble-testsuite (feat/nextjs-blog)
$ git diff -- package.json
diff --git a/package.json b/package.json
index f13c87b..c004c4c 100644
--- a/package.json
+++ b/package.json
@@ -24,7 +24,6 @@
"@mapbox/rehype-prism": "^0.8.0",
"@tailwindcss/typography": "^0.5.0",
"classnames": "^2.3.1",
- "cypress": "^10.1.0",
"gray-matter": "^4.0.2",
"next": "latest",
"next-mdx-remote": "^4.1.0",
@@ -36,6 +35,7 @@
},
"devDependencies": {
"autoprefixer": "^10.4.0",
+ "cypress": "^12.12.0",
"eslint": "<9.0.0",
"eslint-config-next": "12.3.4",
"eslint-config-prettier": "^8.3.0",
gmaca@alpha MINGW64 /e/data/gmaca/github/gmacario/ble-testsuite (feat/nextjs-blog)
$ |
Signed-off-by: Gianpaolo Macario <[email protected]>
See https://github.com/netlify-templates/nextjs-blog-theme#removing-cypress Signed-off-by: Gianpaolo Macario <[email protected]>
Signed-off-by: Gianpaolo Macario <[email protected]>
4563e54
to
4f7f603
Compare
Signed-off-by: Gianpaolo Macario <[email protected]>
Just for test I added a new site on https://app.netlify.com/
and everything worked automagically! Site overview: https://app.netlify.com/sites/ble-testsuite-next/overview Public link: https://ble-testsuite-next.netlify.app/ |
Let us keep the PR open until we complete integrating the existing Markdown pages. and perhaps simplify the theme a little bit. I will adjust the linked Site configuration in Netlify just after the PR is merged. |
✅ Deploy Preview for ble-testsuite-next ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Template available at https://github.com/netlify-templates/nextjs-blog-theme
See demo at https://bejamas-nextjs-blog.netlify.app/