-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
docs: Add sponsors to README #36
Changes from 1 commit
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,34 @@ | ||
name: Data Fetch | ||
|
||
on: | ||
schedule: | ||
- cron: "0 8 * * *" # Every day at 1am PDT | ||
workflow_dispatch: | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Check out repo | ||
uses: actions/checkout@v4 | ||
with: | ||
token: ${{ secrets.WORKFLOW_PUSH_BOT_TOKEN }} | ||
|
||
- name: Set up Node.js | ||
uses: actions/setup-node@v4 | ||
|
||
- name: Install npm packages | ||
run: npm install | ||
|
||
- name: Update README with latest sponsor data | ||
run: npm run build:readme | ||
|
||
- name: Setup Git | ||
run: | | ||
git config user.name "GitHub Actions Bot" | ||
git config user.email "<[email protected]>" | ||
|
||
- name: Save updated files | ||
run: | | ||
chmod +x ./tools/commit-readme.sh | ||
./tools/commit-readme.sh |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
#!/bin/bash | ||
|
||
#------------------------------------------------------------------------------ | ||
# Commits the data files if any have changed | ||
#------------------------------------------------------------------------------ | ||
|
||
if [ -z "$(git status --porcelain)" ]; then | ||
echo "Data did not change." | ||
else | ||
echo "Data changed!" | ||
|
||
# commit the result | ||
git add README.md | ||
git commit -m "docs: Update README sponsors" | ||
|
||
# push back to source control | ||
git push origin HEAD | ||
fi |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
/** | ||
* @fileoverview Script to update the README with sponsors details in all packages. | ||
* | ||
* node tools/update-readme.js | ||
* | ||
* @author Milos Djermanovic | ||
*/ | ||
|
||
//----------------------------------------------------------------------------- | ||
// Requirements | ||
//----------------------------------------------------------------------------- | ||
|
||
import { readFileSync, writeFileSync } from "node:fs"; | ||
import got from "got"; | ||
|
||
//----------------------------------------------------------------------------- | ||
// Data | ||
//----------------------------------------------------------------------------- | ||
|
||
const SPONSORS_URL = | ||
"https://raw.githubusercontent.com/eslint/eslint.org/main/includes/sponsors.md"; | ||
|
||
const README_FILE_PATH = "./README.md"; | ||
|
||
//----------------------------------------------------------------------------- | ||
// Helpers | ||
//----------------------------------------------------------------------------- | ||
|
||
/** | ||
* Fetches the latest sponsors from the website. | ||
* @returns {Promise<string>}} Prerendered sponsors markdown. | ||
*/ | ||
async function fetchSponsorsMarkdown() { | ||
return got(SPONSORS_URL).text(); | ||
} | ||
|
||
//----------------------------------------------------------------------------- | ||
// Main | ||
//----------------------------------------------------------------------------- | ||
|
||
(async () => { | ||
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. Is this IIFE needed? 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. Not really, it just survived through multiple copy-pastes from the original code in eslint/eslint, which is in a cjs module so top-level await was not available. I removed it now. |
||
const allSponsors = await fetchSponsorsMarkdown(); | ||
|
||
// read readme file | ||
const readme = readFileSync(README_FILE_PATH, "utf8"); | ||
|
||
let newReadme = readme.replace( | ||
/<!--sponsorsstart-->[\w\W]*?<!--sponsorsend-->/u, | ||
`<!--sponsorsstart-->\n\n${allSponsors}\n<!--sponsorsend-->`, | ||
); | ||
|
||
// replace multiple consecutive blank lines with just one blank line | ||
newReadme = newReadme.replace(/(?<=^|\n)\n{2,}/gu, "\n"); | ||
|
||
// output to the files | ||
writeFileSync(README_FILE_PATH, newReadme, "utf8"); | ||
})(); |
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.
Suggestion: Can we use an absolute path here? If we're testing locally from the "tools" directory and run this file using a relative path, it may throw an error. An absolute path would provide consistency in all cases, ensuring the file is accessed correctly regardless of the current working directory.
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.
since we are not going to run this manually so using a relative path is also okay.