-
Notifications
You must be signed in to change notification settings - Fork 367
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
Version 4.0.0 #117
Merged
Merged
Version 4.0.0 #117
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added npm badges Updated readme
solved caching issue updated ftp-deploy library
This was
linked to
issues
Nov 13, 2020
Closed
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Version 4.0.0 🚀
Migration Guide
Over the past few months I've been rewriting this action with a brand new architecture that resolves some long standing issues. The new version of this library is heavily inspired by
rsync
.rsync
is a popular application that must be run over assh
connection, it requires thersync
binary be install on both the client and the sever.rsync
provides dozens of advanced arguments that can't be supported due to protocol limitations withftp
. Version 4 attempts to emulate at a high level howrsync
syncs multiple folders, however operates over the constrainedftp
protocol.Less overhead
Version 4 no longer uses docker and has faster startup times!
Improved Defaults
.git
andnode_modules
are now ignored by default, allowing users to opt-in to their deployment instead.Improved Error Handling
Errors are now less cryptic.
Local Testing
Testing actions in version 3 was difficult, often requiring devs to change a value and commit it without testing the change locally. You can now run test configurations using node, see readme for more info.
Minimize downtime
The deployment is now setup in a way that minimizes downtime. For example, new files can be uploaded, then existing files updated, and finally old files deleted.
Removed features 🗑️
sftp
support has been removed. In hindsight this never should have been added. Users withsftp
access should be usingrsync
overssh
. I will create a separate github action for rsync deployments (coming soon)Open Issues Resolved in Version 4 ✅