-
-
Notifications
You must be signed in to change notification settings - Fork 23.1k
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
ci: improve theme preview action #2572
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #2572 +/- ##
=======================================
Coverage 97.29% 97.29%
=======================================
Files 24 24
Lines 4215 4215
Branches 384 384
=======================================
Hits 4101 4101
Misses 112 112
Partials 2 2 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Zo-Bro-23
approved these changes
Mar 6, 2023
j4ckofalltrades
pushed a commit
to j4ckofalltrades/gh-stats
that referenced
this pull request
Mar 6, 2023
tomy0000000
pushed a commit
to tomy0000000/github-readme-stats
that referenced
this pull request
Apr 4, 2023
AlexvZyl
added a commit
to AlexvZyl/github-readme-stats
that referenced
this pull request
Apr 29, 2023
* ci: add e2e vercel test action (anuraghazra#2054) * ci: add e2e vercel test action This commit adds several end-to-end tests that can be used to test whether the Vercel Preview deployment successfully returns the cards. * test: add additional e2e card tests * test: improve e2e card tests This commit makes sure that the tests also check whether a valid cards are returned from the preview deployment. * ci: improve theme-preview actiot push (anuraghazra#2065) This commit makes sure that: - The theme-preview action can test multiple themes. - Fails if something goes wrong. - Requests changes if the theme does not adhere to the guidelines. - Approves the PR if the theme adheres to the guidelines. * ci: improve theme preview action (anuraghazra#2088) This commit makes sure that 4,5,6,8 digit hex colors are also accepted. It further also includes the border_color attribute in the tests. * ci: fix e2e test * chore: Update workflows to cache dependencies (anuraghazra#2083) * ci(label-pr): upgrade actions/labeler to v4 (anuraghazra#2080) Co-authored-by: rickstaa <rick.staa@outlook.com> * ci: fix e2e test * ci: add stale theme pull request closer action push (anuraghazra#2067) This commit adds the `stale-theme-pr-closer` closer action. This action is used to close theme pull requests with an `invalid` label that has been stale for a given number of `STALE_DAYS`. * feat: add transparent theme Adds a transparent theme with a contrast ratio of 4.5 on both black and white (see https://ux.stackexchange.com/a/86226). * feat: remove border color from transparent theme * ci: fix e2e test * feat: added default options to readme (anuraghazra#2051) * feat: added default options to readme * Update readme.md Co-authored-by: Rick Staa <rick.staa@outlook.com> * fix: added default values for stats card * style: some small readme changes Co-authored-by: Rick Staa <rick.staa@outlook.com> * ci: increase top-issues dashboard update frequency * style: improve code style * docs(theme): Auto update theme readme (anuraghazra#2091) Co-authored-by: Github Readme Stats Bot <no-reply@githubreadmestats.com> * ci: fix small bugs in theme-preview action (anuraghazra#2093) * ci: fix small bugs in theme-preview action * ci: format code * ci: fix small bug in theme-preview action * docs: fix link to go India logo svg (anuraghazra#2085) * docs: format readme * docs: fix india care logo * Added Contributors Badge (anuraghazra#2096) * update default values in readme (anuraghazra#2094) * update default values in readme * docs: improve formatting * docs: improve readme punctation Co-authored-by: rickstaa <rick.staa@outlook.com> * fix: adding docstrings to the files where it was missing (anuraghazra#2101) * fix: adding docstrings to missing files * style: format code * style: improve formatting Co-authored-by: Jagruti Tiwari <jagrutit@cdac.in> Co-authored-by: rickstaa <rick.staa@outlook.com> * feat: default values for wakatime and language (anuraghazra#2103) * default values for wakatime and language * docs: update formatting Co-authored-by: rickstaa <rick.staa@outlook.com> * docs: improve readme syntax (anuraghazra#2106) * Update the Vercel deployement guide (anuraghazra#2102) * Update the Vercel deployement guide Updated Readme.md - The guide on deploying your own Vercel instance * docs: fix some small grammer errors Co-authored-by: rickstaa <rick.staa@outlook.com> * Fix card length during error (anuraghazra#2105) * Fix card length during error Fixes anuraghazra#1774 The size of the box should be 550px to allow 25px padding from both sides I found the width of the text using `getComputedTextLength()` method * fix: improve error card size Co-authored-by: rickstaa <rick.staa@outlook.com> * fix: fetch all repos for for the stats card (anuraghazra#2100) * fetch all stars * stop fetching when there are repos with zero stars * remove not needed parameters from the query * add docstring * removed not needed mock * style: update formatting Co-authored-by: rickstaa <rick.staa@outlook.com> * docs: fix readme logo alignment (anuraghazra#2112) * fix: temporary disable multi page star fetch * ci: improve theme preview action This commit makes sure that theme PRs that contain invalid JSON are labeled `invalid` and changes are requested. * style: improve formatting * style: improve small syntax error * Add format script to pre-commit hook (anuraghazra#2146) * Add format script to pre-commit hook * build: add husky prepare step * build: add lint-staged * build: add lint-staged Co-authored-by: rickstaa <rick.staa@outlook.com> * test: fix tests * docs: update readme * docs: add admonition boxes * docs: improve admonition boxes * docs: improve caching documentation * fix: adding doc strings to files in src folder where it was missing (anuraghazra#2129) * fix: adding doc strings to files in src folder where it was missing * refactor: add docstrings * style: run formatter Co-authored-by: rickstaa <rick.staa@outlook.com> * fix: improve graphql error handling * ci: updated issue template (anuraghazra#2127) * Updated issue templates * style: improve template formatting * style: update formatting Co-authored-by: rickstaa <rick.staa@outlook.com> * style: improve templates formatting * style: update formatting * test: burst cache in e2e tests (anuraghazra#2154) * test: increase stats card e2e timeout * test: cleanup tests * ci: add question template * ci: add issues template config and remove question template * fix: prevent errors from being cached (anuraghazra#2160) * test: add cache on error test * docs: add fork update guide * docs: improve syntax * refactor: add extra links to issues template * refactor: improve issues template (anuraghazra#2180) * refactor: improve issues template formatting * test: increase e2e test timeout * build: create npm package (anuraghazra#2151) * build: create npm package This commit makes sure that the common modules of the GRS package can be imported as an npm package. * feat: export card render functions * style: update formatting * fix: fix I18n import error * fix: improve json import (anuraghazra#2190) * Revert "fix: improve json import (anuraghazra#2190)" (anuraghazra#2191) This reverts commit 65424fa. * fix: text overlapped on zh-tw locale (anuraghazra#2192) * ci: fix stale closer action review sort bug (anuraghazra#2201) This commit makes sure the PR staleness is checked by the latest review that is done. * ci: increase invalid theme close time (anuraghazra#2210) This commits increases the theme invalid close time to 20 days. * fix: improve error caching behavoir (anuraghazra#2211) This commit ensures that the GraphQL Pat error is not cached in any browser. See anuraghazra#1471 (comment). * refactor: make issue templates required (anuraghazra#2226) * refactor: fix GitHub capitalization (anuraghazra#2232) * docs: document auto theme change anuraghazra#2074 (anuraghazra#2229) * Document auto theme change in README anuraghazra#2074 * Document auto theme change in README Document auto theme change in README anuraghazra#2074 (update-1) * Document auto theme change updates anuraghazra#2074 * docs: improve dynamic theme documentation Co-authored-by: rickstaa <rick.staa@outlook.com> * Update readme.md * Fix truncation of compact wakatime progress bar when langs_count is set (anuraghazra#2228) * refactor: use default value for languages * fix: truncated compact progress bar * test: add test for truncated compact progress bar with langs_count set * Corrections of Polish expressions in translations.js (anuraghazra#2240) * ci: update theme stale close message * ci: update theme stale close message * Missing term in calculation of TOTAL_VALUES. (anuraghazra#2116) * fix: added missing term in calculation of TOTAL_VALUES. * fix: fix rank test * refactor: update code formatting * refactor: update code formatting Co-authored-by: rickstaa <rick.staa@outlook.com> * Updates the brazilian readme (anuraghazra#2245) * feat: allow users to pass ring_color param (anuraghazra#2075) * fix: prevent crash if undefined color is passed to func fallbackColor * feat: allow users to pass ring_color param Before this commit, the ring color was always equal to the title color. This commit allows the user to pass a ring_color property to allow him to style the ring color the way he pleases to. If no ring_color is passed we default to the title color. * docs: improves ring color documentation * fix: improve code style Co-authored-by: rickstaa <rick.staa@outlook.com> * feat: added Date Night theme to index.js (anuraghazra#2264) * Added Date Night theme to index.js * My theme name should be in snake_case now * test: update snapshots * refactor: migrate to using arrow functions (anuraghazra#2033) * fix: improve boolean parse behaviour (anuraghazra#2029) * refactor: remove unused dotenv calls (anuraghazra#2059) As explained in the Vercel documentation vercel ignores dot files (see vercel/vercel#3962 (comment)). It instead loads the env variables defined in the vercel instance. As a result we can therefore remove dotenv. * fix: fix retry max-out bug (anuraghazra#2121) * fix: fix retry max-out bug This commit makes sure that the retry function tests all PATs. * style: format code * test: fix retry tests * style: format code * feat: improve card loading speed (anuraghazra#2124) * feat: improve card loading times This commit adds the `stale-while-revalidate` option to speed up the card loading times. * mend * feat: add INCLUDE_ORGS env variable (anuraghazra#2275) This commit adds an INCLUDE_ORGS ENV variable that can be used to include results from organizations into the GRS cards for self-deployed Vercel instances. **Warning** This is a hidden feature since we can not officially support this on the Public Vercel instance due to GraphQL and Vercel rate/timeout limits. You can set this env variable via Vercel's ENV variable menu (see https://vercel.com/docs/concepts/projects/environment-variables). * Revert "feat: add INCLUDE_ORGS env variable (anuraghazra#2275)" (anuraghazra#2276) This reverts commit ada9cf4. * fix: fix repoContrib confusion (anuraghazra#2269) (anuraghazra#2274) This commit prevents confusion about the Contributed to stat. Currently, only the last year's results are shown, but it looks like it is the all-time contribution count (see anuraghazra#2269). This commit adds a ' (last year)' suffix to prevent this confusion from happening. * feat: fix 'include_all_commits' inner card margin (anuraghazra#2283) * build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373) Bumps [json5](https://github.com/json5/json5) from 2.2.1 to 2.2.3. - [Release notes](https://github.com/json5/json5/releases) - [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md) - [Commits](json5/json5@v2.2.1...v2.2.3) --- updated-dependencies: - dependency-name: json5 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * test: update snapshots * Add loading Animation to Most used Language card (anuraghazra#2197) * add: text animation * compact progress done * cleaned code * non compact animation done * tests fixed + prettier * revert: vercel.json * remove: unnecessary import * added parameter in docs * style: improve syntax Co-authored-by: rickstaa <rick.staa@outlook.com> * ci: fix theme readme generation action (anuraghazra#2271) * Revert "build(deps): bump json5 from 2.2.1 to 2.2.3 (anuraghazra#2373)" (anuraghazra#2393) This reverts commit 42fdff6. * Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394) This reverts commit 5ae959b. * Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396) This reverts commit 77dcdab. * fix: retryer fixed (anuraghazra#2400) * Revert "Revert "ci: fix theme readme generation action (anuraghazra#2271)" (anuraghazra#2394)" (anuraghazra#2418) This reverts commit 70f0264. * refactor: update code formatting * Revert "Revert "Add loading Animation to Most used Language card (anuraghazra#2197)" (anuraghazra#2396)" (anuraghazra#2419) This reverts commit 4b8198f. * Update readme.md (anuraghazra#2442) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data * feat: fix e2e tests * fix: fix e2e test data * feat: enable multi page star fetching for private vercel instances (anuraghazra#2159) * feat: enable multi-page stars' fetching for private vercel instances This commit enables multi-page stars' support from fetching on private Vercel instances. This feature can be disabled on the public Vercel instance by adding the `FETCH_SINGLE_PAGE_STARS=true` as an env variable in the public Vercel instance. This variable will not be present when people deploy their own Vercel instance, causing the code to fetch multiple star pages. * fix: improve stats multi-page fetching behavoir This commit makes sure that the GraphQL api is only called one time per 100 repositories. The old method added one unnecesairy GraphQL call. * docs: update documentation * style: improve code syntax Co-authored-by: Matteo Pierro <pierromatteo@gmail.com> * lol happy new year * docs: remove rate limit documentation for now Remove the `FETCH_SINGLE_PAGE_STARS` from documentation for now since it might confuse people. * fix: fix error in automatic merge * feat: make sure env variable is read Co-authored-by: Matteo Pierro <pierromatteo@gmail.com> Co-authored-by: Anurag <hazru.anurag@gmail.com> * chore: resolve conflict (anuraghazra#2453) * test: fix e2e tests * test: update snapshots * test: fix e2e test data * test: update e2e test data Co-authored-by: rickstaa <rick.staa@outlook.com> * fix: change prod deployment branch to vercel branch to fix maxDuration bug (anuraghazra#2424) * Create deploy-prep.yml * Create deploy-prep.py * Update vercel.json * Update deploy-prep.yml * Update vercel.json * Added coauthor Co-authored-by: Dou Xiaobo <93511091+douxiaobo@users.noreply.github.com> * Update deploy-prep.yml * refactor: format code * Added if condition to disable deployments on forks Co-authored-by: Rick Staa <rick.staa@outlook.com> * Update deploy-prep.yml Co-authored-by: Dou Xiaobo <93511091+douxiaobo@users.noreply.github.com> Co-authored-by: Anurag Hazra <hazru.anurag@gmail.com> Co-authored-by: rickstaa <rick.staa@outlook.com> * fix: fixes card overflow problem anuraghazra#2452 (anuraghazra#2460) This commit makes sure that the card width is formatted correctly. * ci: prevent certain actions from running on forks (anuraghazra#2466) * feat: add PAT monitoring functions (anuraghazra#2178) * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * feat: add PAT monitoring functions This commit adds two monitoring functions that can be used to check whether the PATs are functioning correctly: - status/up: Returns whether the PATs are rate limited. - status/pat-info: Returns information about the PATs. * feat: add shields.io dynamic badge json response This commit adds the ability to set the return format of the `/api/status/up` cloud function. When this format is set to `shields` a dynamic shields.io badge json is returned. * feat: add 'json' type to up monitor * feat: cleanup status functions * ci: decrease pat-info rate limiting time * feat: decrease monitoring functions rate limits * refactor: pat code * test: fix pat-info tests * Update api/status/pat-info.js Co-authored-by: Anurag Hazra <hazru.anurag@gmail.com> * test: fix broken tests * chore: fix suspended account * chore: simplify and refactor * chore: fix test * chore: add resetIn field --------- Co-authored-by: Anurag <hazru.anurag@gmail.com> * chore: minor changes in pat info (anuraghazra#2481) * ci: add update languages action (anuraghazra#2484) * ci: add update languages action * ci: make sure PR is created when upstream languages are updated * Add `rose` theme (anuraghazra#2480) Co-authored-by: 강민지 <68285922+Kminzzi@users.noreply.github.com> * Support hide_progress for top-langs feature (anuraghazra#2514) * Add support for hide_progress in top languages feature * Fix mistake * Add documents for all languages * Remove unnecessary value check * Update top-languages-card.js * Revert document for all languages except English * Update documentation * Update documentation --------- Co-authored-by: Zohan Subhash <zohan.subhash@gmail.com> * add holi_theme (anuraghazra#2539) * add holi_theme * add holi_theme * Add one_dark_pro (anuraghazra#2507) This colors were taken from One Dark Pro theme in VSCode extention. Please add it and I will use it in my profile Co-authored-by: Zohan Subhash <zohan.subhash@gmail.com> * fix: fix JSON themes bug. (anuraghazra#2544) This fixes a JSON bug that was introduced in anuraghazra#2507. * Add auto-labelling for documentation updates (anuraghazra#2526) * ci: fix unsafe directory bug (anuraghazra#2518) This commit fixes a bug that was introduced due to a upstream change in the git package. See https://stackoverflow.com/questions/71849415/i-cannot-add-the-parent-directory-to-safe-directory-in-git/71904131#71904131 for more information. * Update readme.md (anuraghazra#2414) fix: missing "&" in show_icons=true in Showing icons section * test: update snapshots (anuraghazra#2519) * fix: for issue anuraghazra#2534 (anuraghazra#2536) * Add option to deploy using other services (anuraghazra#2525) * Create express.js * Update readme.md * Update readme.md * ci: update e2e tests (anuraghazra#2548) * ci: fix a bug in the theme preview action (anuraghazra#2549) * refactor: format code (anuraghazra#2550) * refactor: update languages JSON (anuraghazra#2554) Co-authored-by: rickstaa <rickstaa@users.noreply.github.com> * Preview theme workflow fix (anuraghazra#2557) * Fix octokit error * ci: make octokit instance global --------- Co-authored-by: rickstaa <rick.staa@outlook.com> * ci: preview theme workflow fix (anuraghazra#2559) * Fix octokit error * ci: make octokit instance global * Fix preview theme (move declarations to global) * refactor: make constants uppercase --------- Co-authored-by: rickstaa <rick.staa@outlook.com> * refactor: fix code comments and change 'up' rate limit (anuraghazra#2560) * Preview action fix (anuraghazra#2561) * Fix error * refactor: remove unused code --------- Co-authored-by: Rick Staa <rick.staa@outlook.com> * ci: fix theme preview action (anuraghazra#2563) * fix theme preview (anuraghazra#2564) * ci: fix theme preview action * fix: fix some bugs in the 'theme-preveiw' action * ci: fixes theme preview action (anuraghazra#2566) * ci: improve theme preview action (anuraghazra#2572) * Added "Shadow" set (Red, Green, Blue, transparent BG) (anuraghazra#2529) * Added "Shadow" set (Red, Green, Blue, transparent BG) 3 additional themes sticking primarily to flat colors, which the exception of icons and border being slightly darker. All 3 themes also have transparent backgrounds that will show differently per-user via GiHub's own light and dark themes. Transparency should also still provide easy readability for both. * Test Just want to see if we can make the themes have a transparent background. * Shadows moved under Transparent --------- Co-authored-by: Zohan Subhash <zohan.subhash@gmail.com> * ci: fix theme docs generate bug (anuraghazra#2573) * update my theme (anuraghazra#2576) * refactor: update languages JSON (anuraghazra#2596) Co-authored-by: rickstaa <rickstaa@users.noreply.github.com> * Add format stats option (anuraghazra#2155) * feat: added `format_stats` option (anuraghazra#2128) * refactor: change `format_stats` to `short_values` (anuraghazra#2128) * test: create shorten values test (anuraghazra#2128) * Update readme.md Co-authored-by: Rick Staa <rick.staa@outlook.com> * refactor: rename ``short_values`` to ``number_format`` * Update readme.md Co-authored-by: Rick Staa <rick.staa@outlook.com> * Update src/cards/stats-card.js Co-authored-by: Rick Staa <rick.staa@outlook.com> * refactor: format codebase --------- Co-authored-by: Rick Staa <rick.staa@outlook.com> * add github_dark_dimmed theme (anuraghazra#2594) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank_icon option (anuraghazra#2628) * feat(theme): add github_dark_dimmed theme * feat(theme): change github_dark_dimmed icon color * contrast ratio adjustment contrast ratio adjustment on github_dark_dimmed theme * feat(theme): readme preview * feat(theme): github themes next to each other * github themes next to each other * feat(RankIcon): add rank icon option * feat(RankIcon): extract rankIcon to icons file * feat(RankIcon): update readme * feat(RankIcon): test coverage * Update readme.md Co-authored-by: Rick Staa <rick.staa@outlook.com> --------- Co-authored-by: Rick Staa <rick.staa@outlook.com> * Updated the custom error there was a Typo. (anuraghazra#2618) "Something went while trying to retrieve the stats data using the GraphQL API." -> "Something went wrong while trying to retrieve the stats data using the GraphQL API." * Fix typos inside wakatime test name (anuraghazra#2617) * removed redundant comparison (anuraghazra#957) * dev (anuraghazra#1782) * fix path to powered by vercel image (anuraghazra#1792) * Synchonize cache seconds min value inside docs translations with main readme (anuraghazra#2616) * New top language algorithm implementation (anuraghazra#1732) * Reduced vercel maxDuration * Implemented new algorithm for Top Langs * Revert "Reduced vercel maxDuration" This reverts commit b0bc626. * Added documentation * Fixed broken implementation * Update fetchTopLanguages.test.js Changed tests * Now uses the general formula The parameters p and q can be set by the user. * Updated tests and added new test * Added new test New test for order by repo count. * Updated documentation Added explanation and examples for new options. * Updated documentation This was overwritten in the merge commit. * docs: improve docs and fix tests * Renamed parameters Renamed `p` and `q` to `size_weight` and `count_weight`, respectively. * Updated the documentation Changes introduced in f2516d6 --------- Co-authored-by: rickstaa <rick.staa@outlook.com> * feat: improve CONTRIBUTING.md (anuraghazra#2609) * docs: update give logo (anuraghazra#2605) * Fixed typo in word color inside README (anuraghazra#2652) * Fixed docs typo inside src/common/utils.js (anuraghazra#2651) * Fixed several typos inside preview theme script (anuraghazra#2650) * Fixed todo inside stats card data fetcher (anuraghazra#2649) * docs: fix README language card usage order (anuraghazra#2659) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Rick Staa <rick.staa@outlook.com> Co-authored-by: Jongwoo Han <han980817@gmail.com> Co-authored-by: Oscar Dominguez <dominguez.celada@gmail.com> Co-authored-by: Jagruti Tiwari <tiwarijagruti1@gmail.com> Co-authored-by: Anurag Hazra <hazru.anurag@gmail.com> Co-authored-by: Github Readme Stats Bot <no-reply@githubreadmestats.com> Co-authored-by: ianbromwichuos <112479962+ianbromwichuos@users.noreply.github.com> Co-authored-by: Manan1612 <mananpatel161200@gmail.com> Co-authored-by: Ayushi Vishwakarma <ayushivishwakarma22@gmail.com> Co-authored-by: Jagruti Tiwari <jagrutit@cdac.in> Co-authored-by: Aditya Shelke <103348863+adi-uchiha@users.noreply.github.com> Co-authored-by: Akshat Goel <98082704+DecryptAG@users.noreply.github.com> Co-authored-by: nekiwo <58634064+nekiwo@users.noreply.github.com> Co-authored-by: Matteo Pierro <pierromatteo@gmail.com> Co-authored-by: Taehyun Hwang <eeht1717@gmail.com> Co-authored-by: Shivam Kotak <shivamkotak16.sk@gmail.com> Co-authored-by: Brian Choromanski <BrianChoromanski@gmail.com> Co-authored-by: Kurt <32745146+kurt-liao@users.noreply.github.com> Co-authored-by: Frieder Bluemle <frieder.bluemle@gmail.com> Co-authored-by: Falguni Sarkar <falgunisarkar526@gmail.com> Co-authored-by: Ray Chan <ray25222@gmail.com> Co-authored-by: Tymoteusz Marzec <59064153+InfoTCube@users.noreply.github.com> Co-authored-by: Fabian Joswig <fabian.joswig@ed.ac.uk> Co-authored-by: Ítalo Sousa <italusousa@gmail.com> Co-authored-by: Pranav Joglekar <pranav2000joglekar@gmail.com> Co-authored-by: Bradley Jester <109996512+jesterb0206@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Huzaifa Khilawala <55500003+RedHeadphone@users.noreply.github.com> Co-authored-by: Zohan Subhash <zohan.subhash@gmail.com> Co-authored-by: Dou Xiaobo <93511091+douxiaobo@users.noreply.github.com> Co-authored-by: 강민지 <68285922+Meezzi@users.noreply.github.com> Co-authored-by: 강민지 <68285922+Kminzzi@users.noreply.github.com> Co-authored-by: Amir <amirhhakimnejad@gmail.com> Co-authored-by: Mohamed Hassan <mo.hassan2792@gmail.com> Co-authored-by: Oleksandr Perlov <oleksandrperlov@gmail.com> Co-authored-by: Cateline Mnemosyne <123184375+catelinemnemosyne@users.noreply.github.com> Co-authored-by: Raphael Gonçalves <89359384+raphaelricardo10@users.noreply.github.com> Co-authored-by: Rehman <rehman.2468@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: rickstaa <rickstaa@users.noreply.github.com> Co-authored-by: Etanarvazac Revorix <etanarvazac@outlook.com> Co-authored-by: Eduardo Zaniboni <67515606+eduardozaniboni@users.noreply.github.com> Co-authored-by: Caeden Perelli-Harris <caedenperelliharris@gmail.com> Co-authored-by: Fabiano Couto <facodeli@gmail.com> Co-authored-by: LOKESH SINGH <77314004+LokeshXs@users.noreply.github.com> Co-authored-by: Alexandr Garbuzov <53787217+qwerty541@users.noreply.github.com> Co-authored-by: kitswas <90329875+kitswas@users.noreply.github.com>
devantler
pushed a commit
to devantler/github-readme-stats
that referenced
this pull request
May 1, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Another try. The following code blocks can be improved, but I currently do not have the time to look at them:
github-readme-stats/scripts/preview-theme.js
Lines 315 to 342 in 78d345d
This fixes some differences in syntax between
JS
andJSON
. There is probably a better way to do this (i.e. external library, better regex or changethemes.js
tothemes.json
).https://github.com/anuraghazra/github-readme-stats/blob/78d345d30e7df37d4378f829cdde4820d0cc967b/scripts/preview-theme.js#L405C6-L413
This code gets the additions from the git diff. This code was initially added by @anuraghazra, but I quickly updated it to work with multiple theme additions.
Tests