Skip to content

🔧 Update rollup plugin dependencies to rollup-preserve-directives#86

Merged
yceffort-naver merged 3 commits into
mainfrom
feature/preserve-directives
May 23, 2025
Merged

🔧 Update rollup plugin dependencies to rollup-preserve-directives#86
yceffort-naver merged 3 commits into
mainfrom
feature/preserve-directives

Conversation

@yceffort-naver

Copy link
Copy Markdown
Contributor

Related Issue

Describe your changes

Request

@yceffort-naver yceffort-naver self-assigned this May 14, 2025
@yceffort-naver

Copy link
Copy Markdown
Contributor Author

/canary-publish

@npayfebot

npayfebot commented May 14, 2025

Copy link
Copy Markdown
Contributor

✅ Changeset detected

Latest commit: 6f6b852

@naverpay/pite package have detected changes.

If no version change is needed, please add skip-detect-change to the label.

The changes in this PR will be included in the next version bump.

powered by: naverpay changeset detect-add actions

@npayfebot

Copy link
Copy Markdown
Contributor

Please specify the detect version for a valid canary version deployment

@npayfebot

npayfebot commented May 14, 2025

Copy link
Copy Markdown
Contributor

NPM Packages

📦 @naverpay/pite

🧩 Dependency Changes

Package Status Previous Version Updated Version Bundle Size (Max) Changed
rollup-preserve-directives@^1.1.3 - ^1.1.3 19.2 kB 19.2 kB
rollup-plugin-preserve-directives@^0.4.0 ^0.4.0 - -23.4 kB -23.4 kB

powered by: naverpay size-action

@yceffort-naver

Copy link
Copy Markdown
Contributor Author

/canary-publish

1 similar comment
@yceffort-naver

Copy link
Copy Markdown
Contributor Author

/canary-publish

@yceffort-naver

Copy link
Copy Markdown
Contributor Author

/canary-publish

1 similar comment
@yceffort-naver

Copy link
Copy Markdown
Contributor Author

/canary-publish

@npayfebot

Copy link
Copy Markdown
Contributor

Published Canary Packages

@naverpay/pite@2.2.1-canary.250514-6f6b852

@2-one-week 2-one-week left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@yceffort-naver

Copy link
Copy Markdown
Contributor Author

I switched to rollup-preserve-directives because it has more users and is more stable compared to rollup-plugin-preserve-directives.

@yceffort-naver yceffort-naver merged commit 0165187 into main May 23, 2025
4 checks passed
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.

3 participants