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

Fix go#template#create() for directories that don't exist yet #1618

Merged
merged 1 commit into from
Dec 22, 2017
Merged

Fix go#template#create() for directories that don't exist yet #1618

merged 1 commit into from
Dec 22, 2017

Conversation

arp242
Copy link
Contributor

@arp242 arp242 commented Dec 17, 2017

Previously vim ~/go/src/newdir/main.go would result in:

"~/go/src/newdir/main.go" [New DIRECTORY]
Error detected while processing function <SNR>56_template_autocreate[3]..go#template#create:
line    6:
E344: Can't find directory "/home/martin/go/src/newdir" in cdpath
E472: Command failed

With this change this gets treated like a directory which isn't a Go
package yet.

@codecov-io
Copy link

codecov-io commented Dec 17, 2017

Codecov Report

Merging #1618 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1618      +/-   ##
==========================================
+ Coverage   19.15%   19.19%   +0.03%     
==========================================
  Files          53       53              
  Lines        4151     4153       +2     
==========================================
+ Hits          795      797       +2     
  Misses       3356     3356
Flag Coverage Δ
#nvim 14.54% <100%> (+0.04%) ⬆️
#vim74 17.57% <100%> (+0.03%) ⬆️
#vim80 18.34% <100%> (+0.03%) ⬆️
Impacted Files Coverage Δ
autoload/go/template.vim 64.7% <100%> (+2.2%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60a03b1...22f2c0d. Read the comment docs.

Repository owner deleted a comment from bhcleek Dec 18, 2017
Repository owner deleted a comment from bhcleek Dec 18, 2017
Previously `vim ~/go/src/newdir/main.go` would result in:

	"~/go/src/newdir/main.go" [New DIRECTORY]
	Error detected while processing function <SNR>56_template_autocreate[3]..go#template#create:
	line    6:
	E344: Can't find directory "/home/martin/go/src/newdir" in cdpath
	E472: Command failed

With this change this gets treated like a directory which isn't a Go
package yet.
@arp242 arp242 merged commit 3a8cc93 into fatih:master Dec 22, 2017
@arp242 arp242 deleted the tpldir branch December 22, 2017 13:01
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