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

Link untracked files #113

Closed
wants to merge 2 commits into from

Conversation

andresilva
Copy link
Contributor

This is an option to link untracked files by git while still ignoring files that are git ignored. The motivation for this is that I have some dotfiles that are untracked by git since they contain private information but which I still want to link.
Also, when I'm working on my emacs configuration I sometimes have new files that I don't want to track right away but there's no way to link them easily since they're inside a submodule.

@andresilva
Copy link
Contributor Author

Ping.

@andsens
Copy link
Owner

andsens commented Oct 5, 2014

Hey, sorry about the delay. I'm in the middle of a move so I haven't had the time to sit down and have a proper look at it (also: there was a bug which I needed to prioritize over your PR).
Hopefully I'll find the time in a week or so.

@andsens
Copy link
Owner

andsens commented Oct 12, 2014

Have a look at #117 and #108. It seems the linking function needs a bit of work after we changed it to use git ls-files.
I'd like to get the bugs out of the way before we add new features.

The change definitely makes sense (good use-case descriptions btw.) and will make it into homeshick, but I think it would be prudent to wait until the linking code is a little more stable.

I am closing this PR, but I created #118. Now it's on the todo list and we'll get to it once the bugs are squashed.

@andsens andsens closed this Oct 12, 2014
@andresilva
Copy link
Contributor Author

I agree that fixing the issues with the current implementation is more important. Cheers!

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.

2 participants