Skip to content

Conversation

@james-pre
Copy link

This PR updates @zenfs/core and @zenfs/dom to stable versions (i.e. ^1.0.0) and updates @zenfs/zip to the latest minor release (^0.5.0).

Copy link
Contributor

@niedzielski niedzielski left a comment

Choose a reason for hiding this comment

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

thank you so much, @james-pre! I left a little feedback inline 👍 basic editor smoketest looks good but I'm flagging @ObsidianSnoo to check the asset functionality.

@james-pre
Copy link
Author

Just FYI, @zenfs/zip has been merged into @zenfs/archives and is now deprecated.

@Krakabek Krakabek mentioned this pull request Jan 6, 2025
3 tasks
ObsidianSnoo added a commit that referenced this pull request Jan 6, 2025
<!-- If this pull request closes an issue, please mention the issue
number below -->
Closes # <!-- Issue # here -->

## 💸 TL;DR
<!-- What's the three sentence summary of purpose of the PR -->
Migrates the unsupported dependency, follow up for
#40

## 📜 Details
[Design Doc](<!-- insert Google Doc link here if applicable -->)

[Jira](<!-- insert Jira link if applicable -->)

<!-- Add additional details required for the PR: breaking changes,
screenshots, external dependency changes -->

## 🧪 Testing Steps / Validation
<!-- add details on how this PR has been tested, include reproductions
and screenshots where applicable -->

## ✅ Checks
<!-- Make sure your pr passes the CI checks and do check the following
fields as needed - -->
- [x] CI tests (if present) are passing
- [x] Adheres to code style for repo
- [ ] Contributor License Agreement (CLA) completed if not a Reddit
employee

---------

Co-authored-by: danil.radkovskyi <danil.radkovskyi@reddit.com>
Co-authored-by: ObsidianSnoo <66971815+ObsidianSnoo@users.noreply.github.com>
@james-pre james-pre closed this by deleting the head repository Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants