-
Notifications
You must be signed in to change notification settings - Fork 0
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
Renewal v1 #44
base: main
Are you sure you want to change the base?
Renewal v1 #44
Changes from 19 commits
a4e4378
dcaed86
d9f3276
57570aa
11afdaa
5b905b1
371052a
f2eee84
fbcb925
966558f
8917231
39cad0f
c86549f
0d6ed5f
7bd5f89
9f680a8
d09690d
834db36
c19d982
e96bcc6
ae2ec1b
d1813c5
dda2478
43c4241
c55b2e8
dd5e4fb
d2c654c
b721584
42e15d0
1e4041b
72f723a
5793d4c
a5c38da
ad2a12a
955d26e
69aed6d
1fc0bed
ca477a9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
root = true | ||
|
||
[*] | ||
indent_style = tab | ||
tab_width = 2 | ||
|
||
[{src,scripts}/**.{ts,json,js,cjs}] | ||
end_of_line = crlf | ||
charset = utf-8 | ||
trim_trailing_whitespace = true | ||
insert_final_newline = true | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
*.mdx | ||
*.md |
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -1,23 +1,24 @@ | ||||||||||
import { defineConfig } from "astro/config"; | ||||||||||
import { vanillaExtractPlugin } from "@vanilla-extract/vite-plugin"; | ||||||||||
import remarkBreaks from "remark-breaks"; | ||||||||||
import mdx from "@astrojs/mdx"; | ||||||||||
import sitemap from "@astrojs/sitemap"; | ||||||||||
|
||||||||||
// https://astro.build/config | ||||||||||
import codeBlockPlugin from "./tools/remark-code-quote"; | ||||||||||
import wrapH2WithSection from "./tools/rehype-wrap-h2-with-section"; | ||||||||||
import { astroLastModifiedAt } from "./tools/remark-astro-last-modified-at"; | ||||||||||
Comment on lines
+5
to
+7
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Imported modules are missing in the The files
🔗 Analysis chainLGTM! Verify the imported modules. The new imports for custom plugins look good. They follow a consistent naming convention and are appropriately imported from local files. To ensure the imported modules exist and are correctly implemented, run the following script: 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Verify the existence and basic structure of the imported modules
# Test: Check if the files exist and contain expected content
for file in "tools/remark-code-quote.js" "tools/rehype-wrap-h2-with-section.js" "tools/remark-astro-last-modified-at.js"; do
if [ -f "$file" ]; then
echo "File $file exists."
grep -q "export default" "$file" && echo " - Contains default export." || echo " - Warning: No default export found."
else
echo "Warning: File $file does not exist."
fi
done
Length of output: 923 |
||||||||||
|
||||||||||
// https://astro.build/config | ||||||||||
export default defineConfig({ | ||||||||||
// site: 'https://example.com', | ||||||||||
site: "https://wagomu.me", | ||||||||||
integrations: [mdx(), sitemap()], | ||||||||||
integrations: [mdx(), sitemap(), tailwind()], | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add Tailwind import. The addition of Add the following import at the top of the file: +import tailwind from "@astrojs/tailwind"; 📝 Committable suggestion
Suggested change
|
||||||||||
vite: { | ||||||||||
plugins: [vanillaExtractPlugin()], | ||||||||||
plugins: [] | ||||||||||
}, | ||||||||||
markdown: { | ||||||||||
remarkPlugins: [remarkBreaks], | ||||||||||
remarkPlugins: [remarkBreaks, codeBlockPlugin, astroLastModifiedAt], | ||||||||||
rehypePlugins: [wrapH2WithSection], | ||||||||||
shikiConfig: { | ||||||||||
theme: "github-light", | ||||||||||
}, | ||||||||||
}, | ||||||||||
theme: "github-light" | ||||||||||
} | ||||||||||
} | ||||||||||
}); |
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.
🛠️ Refactor suggestion
Consider using LF instead of CRLF for better cross-platform compatibility.
The specific settings for .ts, .json, .js, and .cjs files in the src and scripts directories are generally good:
However, the use of CRLF line endings may cause issues on non-Windows systems. Consider using LF instead for better cross-platform compatibility.
Apply this change to use LF line endings:
📝 Committable suggestion