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

Install Packages post release #482

Merged
merged 33 commits into from
Dec 11, 2024
Merged

Conversation

aaronmgdr
Copy link
Member

@aaronmgdr aaronmgdr commented Dec 10, 2024

After plackages are released, attempt to install each of them. combined with prereleasing this should help us catch any issues arising from packages being used outside of development environment

tested

run with mock data
https://github.com/celo-org/developer-tooling/actions/runs/12275055520/job/34249530769?pr=482


PR-Codex overview

This PR updates the GitHub Actions workflows and modifies the package.json file for a project, primarily focusing on changing Node.js versions, enhancing package publishing processes, and improving the installation of released packages.

Detailed summary

  • Deleted .github/workflows/cron-npm-install.yml.gitignore.
  • Updated node-version from 18.x to 20.x in the release.yaml workflow.
  • Changed prepack script in package.json to use yarn run build.
  • Added new steps for formatting output and installing released packages in the workflow.
  • Adjusted conditions for opening documentation PRs to check for @celo/celocli.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented Dec 10, 2024

⚠️ No Changeset found

Latest commit: c46a27b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

codecov bot commented Dec 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.44%. Comparing base (b83d8c4) to head (42def9e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #482   +/-   ##
=======================================
  Coverage   73.44%   73.44%           
=======================================
  Files         240      240           
  Lines       10180    10180           
  Branches     1607     1614    +7     
=======================================
  Hits         7477     7477           
- Misses       2612     2622   +10     
+ Partials       91       81   -10     
Components Coverage Δ
celocli 85.23% <ø> (ø)
sdk 68.94% <ø> (ø)
wallets 52.72% <ø> (ø)
viem-sdks 95.08% <ø> (ø)

@aaronmgdr aaronmgdr changed the title Aaronmgdr/release ci improvements test Install Packages post release Dec 11, 2024
@aaronmgdr aaronmgdr marked this pull request as ready for review December 11, 2024 11:12
@aaronmgdr aaronmgdr requested a review from a team as a code owner December 11, 2024 11:12
@aaronmgdr aaronmgdr force-pushed the aaronmgdr/release-ci-improvements-test branch from d4cf677 to c46a27b Compare December 11, 2024 11:12
.github/workflows/release.yaml Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/release.yaml Show resolved Hide resolved
@aaronmgdr aaronmgdr merged commit ef9ca97 into master Dec 11, 2024
27 of 50 checks passed
@aaronmgdr aaronmgdr deleted the aaronmgdr/release-ci-improvements-test branch December 11, 2024 14:09
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.

2 participants