Skip to content

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

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

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

Conversation

@yceffort-naver
Copy link
Contributor

Related Issue

Describe your changes

Request

@yceffort-naver yceffort-naver self-assigned this May 14, 2025
@yceffort-naver
Copy link
Contributor Author

/canary-publish

@npayfebot
Copy link
Contributor

npayfebot commented May 14, 2025

✅ 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
Contributor

Please specify the detect version for a valid canary version deployment

@npayfebot
Copy link
Contributor

npayfebot commented May 14, 2025

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
Contributor Author

/canary-publish

1 similar comment
@yceffort-naver
Copy link
Contributor Author

/canary-publish

@yceffort-naver
Copy link
Contributor Author

/canary-publish

1 similar comment
@yceffort-naver
Copy link
Contributor Author

/canary-publish

@npayfebot
Copy link
Contributor

Published Canary Packages

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

Copy link
Member

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

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
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