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

ci: switch to npm ci in github actions #2046

Merged
merged 2 commits into from Sep 24, 2022
Merged

ci: switch to npm ci in github actions #2046

merged 2 commits into from Sep 24, 2022

Conversation

rickstaa
Copy link
Collaborator

Use npm ci nstead of npm install in actions (see https://docs.npmjs.com/cli/v8/commands/npm-ci).

@vercel
Copy link

vercel bot commented Sep 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
github-readme-stats ✅ Ready (Inspect) Visit Preview Sep 24, 2022 at 8:36AM (UTC)

@codecov
Copy link

codecov bot commented Sep 24, 2022

Codecov Report

Base: 94.66% // Head: 94.66% // No change to project coverage 👍

Coverage data is based on head (864b797) compared to base (0de23d3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2046   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files          22       22           
  Lines         712      712           
  Branches      219      219           
=======================================
  Hits          674      674           
  Misses         33       33           
  Partials        5        5           

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

@rickstaa rickstaa merged commit c7d4c29 into master Sep 24, 2022
@rickstaa rickstaa deleted the switch_to_npm_ci branch September 24, 2022 08:37
infinity-plus pushed a commit to infinity-plus/github-readme-stats that referenced this pull request Sep 24, 2022
devantler pushed a commit to devantler/github-readme-stats that referenced this pull request Sep 26, 2022
Nagico added a commit to Nagico/github-readme-stats that referenced this pull request Oct 9, 2022
* Fixed misspelling (anuraghazra#1885)

* docs: incorrect cache documentation (anuraghazra#1889)

* docs: note for temporary cache inc (anuraghazra#1890)

* build: pin node version to >=13 (anuraghazra#1900) (anuraghazra#1901)

This commit pins the node version to >=13 since node 12 is now end of
life (see https://vercel.com/changelog/node-js-12-is-being-deprecated).

* refactor: improve issues template

Move FAQ notice to the top of the issues template to increase the change of users reading it.

* refactor: fix bug in issues template

* refactor: improve issues template

Move part of the FAQ notice to the bottom so that it doesn't clutter the issues window.

* refactor: fix typo in comment (anuraghazra#1912)

* refactor: remove arrow function body braces (anuraghazra#1917)

* refactor: update license year (anuraghazra#1735)

* Updated Year

* Update LICENSE / Revert to 2020

The MIT license [year] field should indicate the year the copyright was applied

Co-authored-by: Rick Staa <rick.staa@outlook.com>

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* refactor: improve level calculator (anuraghazra#1946)

* test: fix gradient test by using more specific querySelector (anuraghazra#1954)

* feat: add min width languages card (anuraghazra#1953)

* Add min width to languages card

* Add test for min width languages card

* feat: increase lang card min width

Co-authored-by: Rick Staa <rick.staa@outlook.com>

* ci: add top-issues-dashboard action

This commit adds the top issues dashboard action which creates a issue displaying the top issues of the week.

* ci: fix title of top-issues action

* ci: fix top-issues-action syntax (anuraghazra#1978)

* ci: add top-issues-action argument

* Fix NaN and undefined for empty profiles (anuraghazra#1965)

* totalCommits: don't return NaN

* rank: B+ should cover everyone

Empty profile used to show "undefined" as the rank. Now empty profile shows "A+"... is B+ possible?

* Remove unneeded underscore from Readme (anuraghazra#2003)

* Add prettier check to CI (anuraghazra#1999)

* Add prettier check to CI

* Run prettier:format

* ci: add empty issues closer action (anuraghazra#2004)

Co-authored-by: Prashubh Atri <prashubh27@gmail.com>

Co-authored-by: Prashubh Atri <prashubh27@gmail.com>

* Fix anuraghazra#1170: Allow customizable width for GitHub Stats Card (anuraghazra#1334)

* Change default stats card width with hide rank

* Add tests for stats card with card_width

* Add card_width Stats Card description to readme

* fix: add icon width to stats-card min width calculation

* fix: fixes rank circle padding problem

This commit fixes a padding problem that was introduced in
f9c0e0b. In the new code, the padding
around the rank circle will be 50 when the stats card is bigger than
450. When it is smaller than 450 the left and right padding will shrink
equally.

* style: run prettier

* tests: add extra stats 'card_width' tests

This commit makes sure we also test the stats card width for the case that the 'show_icons'
option is enabled.

* style: run prettier

Co-authored-by: rickstaa <rick.staa@outlook.com>

* refactor: change formatting command

* feat: add 'exclude_repo' option to stats card (anuraghazra#1450)

This commit allows users to exclude repositories for the stats card
using the `exclude_repo` option.

* fix: add missing StatsCard types

* feat(stats): add 'not_bold' option to stats card (anuraghazra#1468)

* ci: change theme-preview action run condition (anuraghazra#1903)

This commit makes sure that the theme-preview action only runs when a
theme was added or changed.

* Feature/grs 1955 change commonjs imports (anuraghazra#1995)

* GRS-1955: Using ES6 import/export in src files

* GRS-1955: Using ES6 import/export in test files

* GRS-1955: Using ES6 import/export in themes index.js

* GRS-1955: Readding blank line at end of top-languages-card.js

* feat: fix test es6 import errors

This commit makes sure jest is set-up to support es6. It also fixes
several test errors and sorts the imports.

* test: update test node version

This commit makes sure node 16 is used in the github actions.

* refactor: run prettier

Co-authored-by: rickstaa <rick.staa@outlook.com>

* fix: fix generate-theme-doc bug

* ci: switch to npm ci in github actions (anuraghazra#2046)

* fix: fix es6 import file extension

* refactor: clean up imports

* build: update package-lock file

* build: fix wakatime json import bug (anuraghazra#2050)

* build: fix vercel node version

* build: remove engine property

* fix: add json import workarround

This commit adds a workaround for importing the `languageColors` JSON
file. This needed to be done since Vercel uses v16.4, which does not
support JSON file importing without the `experimental-json-modules`
flag. See https://simonplend.com/import-json-in-es-modules/
for more information. The workaround can be found [here](https://stackoverflow.com/questions/66726365/how-should-i-import-json-in-node).

* ci: filter anuraghazra#1772 from top-issues-dashboard

* 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

Co-authored-by: Fidel Alejandro Fernández Arias <47539231+ale94lko@users.noreply.github.com>
Co-authored-by: tmt <101849050+tinnamchoi@users.noreply.github.com>
Co-authored-by: Rick Staa <rick.staa@outlook.com>
Co-authored-by: Cong <congusbongus@gmail.com>
Co-authored-by: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com>
Co-authored-by: Daniel <danopperm@gmail.com>
Co-authored-by: akmhmgc <38002468+akmhmgc@users.noreply.github.com>
Co-authored-by: Qi Mo Lin <60514926+qimolin@users.noreply.github.com>
Co-authored-by: Adil Hanney <adilh+github@orchid.anonaddy.com>
Co-authored-by: John Sweeney <email@sarno.dev>
Co-authored-by: Steven <s3@uwaterloo.ca>
Co-authored-by: Prashubh Atri <prashubh27@gmail.com>
Co-authored-by: Artem Kostiuk <postatum@gmail.com>
Co-authored-by: rsk2 <beingrohan24.7@gmail.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>
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.

None yet

1 participant