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

Map local to remote directories. (specific upload locations) #51

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

TimVosch
Copy link

Hey, I added support to map local directories to remote directories.

For example, if I have a folder build and I want it to upload to my remote index instead of a build folder I'd do:

"directorymap": {
        "build": ""
    }

If I'd want to map my local test folder to a remote beta folder, i can do:

"directorymap": {
        "test": "beta"
    }

When you select local to remote and then directory test it will automatically upload to folder beta instead of test if this is specified in the directorymap

@TimVosch
Copy link
Author

I just realized it curently only works if you upload the whole directory.

If I have dist mapped to build and I where to upload a folder dist/css then it would still upload as dist/css and not build/css. This is not hard to change. I'll add it once I have some time.

@massimocode
Copy link

@TimVosch This would be a great thing to add. Did you have a chance to look into the issue you noticed with your original PR? If we could get this in that would be great. Let me know if you need a hand.

@hajekj
Copy link
Collaborator

hajekj commented Nov 1, 2016

@TimVosch Hello, due to recent merges, this PR has conflicts with merging currently. Could you please take a look and fix the conflicts so we can merge the change? Thanks a lot!

@rabbishuki
Copy link

This would a really good feature, please do it..

@edimitchel
Copy link

PLEASE, merge this PR !

@condorman
Copy link

PLEASE merge, i need this feature thx!

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.

6 participants