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: remove scripts.postinstall in CI scripts #130

Merged
merged 3 commits into from
Nov 21, 2022
Merged

ci: remove scripts.postinstall in CI scripts #130

merged 3 commits into from
Nov 21, 2022

Conversation

motss
Copy link
Owner

@motss motss commented Nov 21, 2022

Description

This fixes #129.

Changes

  1. Remove unnecessary scripts in package.json for releases in CI scripts

Signed-off-by: Rong Sen Ng (motss) <wes.ngrongsen@gmail.com>
@motss motss self-assigned this Nov 21, 2022
Signed-off-by: Rong Sen Ng (motss) <wes.ngrongsen@gmail.com>
Signed-off-by: Rong Sen Ng (motss) <wes.ngrongsen@gmail.com>
@codecov
Copy link

codecov bot commented Nov 21, 2022

Codecov Report

Merging #130 (59b431a) into main (5f96d66) will not change coverage.
The diff coverage is n/a.

❗ Current head 59b431a differs from pull request most recent head 7cb11a4. Consider uploading reports for the commit 7cb11a4 to get more accurate results

@@            Coverage Diff            @@
##              main      #130   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            5         5           
  Lines          152       152           
  Branches         5         5           
=========================================
  Hits           152       152           
Flag Coverage Δ
unit_tests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@motss motss merged commit d593ffb into main Nov 21, 2022
@motss motss deleted the fix/129 branch November 21, 2022 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: postinstall fails in pnpm-monorepo
1 participant