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

fix: publish pillarbox-suite to npm #26

Merged
merged 1 commit into from
Jul 2, 2024
Merged

fix: publish pillarbox-suite to npm #26

merged 1 commit into from
Jul 2, 2024

Conversation

jboix
Copy link
Contributor

@jboix jboix commented Jul 2, 2024

Description

Automated the process of publishing pillarbox-suite packages to npm by updating necessary configurations and workflows.

Changes Made

  • Updated GitHub Actions workflows to automate the publishing process to npm.
  • Modified package.json files to ensure correct npm configuration.
  • Added a LICENSE file to each package and updated the create script to automatically include one.

Checklist

  • I have followed the project's style and contribution guidelines.
  • I have performed a self-review of my own changes.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.

@jboix jboix requested a review from amtins July 2, 2024 12:16
Copy link

github-actions bot commented Jul 2, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 94.28% 956 / 1014
🔵 Statements 94.28% 956 / 1014
🔵 Functions 94.2% 65 / 69
🔵 Branches 91.34% 95 / 104
File CoverageNo changed files found.
Generated in workflow #79

@jboix jboix self-assigned this Jul 2, 2024
@jboix jboix marked this pull request as ready for review July 2, 2024 12:17
Copy link

github-actions bot commented Jul 2, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-07-02 12:38 UTC

@jboix jboix force-pushed the fix/npm-publish branch from ed2ce16 to 94aa4d7 Compare July 2, 2024 12:23
Copy link
Member

@amtins amtins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Automated the process of publishing pillarbox-suite packages to npm by updating necessary
configurations and workflows.

Changes include:

- Updated GitHub Actions workflows to automate the publishing process to npm.
- Modified package.json files to ensure correct npm configuration.
- Added a LICENSE file to each package and updated the `create` script to automatically include one.
@jboix jboix force-pushed the fix/npm-publish branch from 94aa4d7 to 0bceebc Compare July 2, 2024 12:25
@jboix jboix merged commit 74e7ada into main Jul 2, 2024
4 checks passed
@jboix jboix deleted the fix/npm-publish branch July 2, 2024 12:38
Copy link

github-actions bot commented Jul 2, 2024

🎉 This PR is included in version @srgssr/pillarbox-playlist-v1.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

github-actions bot commented Jul 2, 2024

🎉 This PR is included in version @srgssr/skip-button-v1.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants