-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
resolve watch issues with empty siteContents; remove unneeded Option #103
Conversation
I was looking forward to trying this, but had to downgrade my targeting packs first (*). (Are we sure that (*) Linux does clean dotnet installs by default, so a simple feature upgrade leaves
The "solution" (for deb-based distros):
|
@rdipardo i just tested it locally using |
That works! Bravo! |
I plan to test this week on a file deletion. That might be the only thing left to do? |
Please do! |
Is there something else that needs to be tested before this could be merged? |
I tested it with deletes: it removes any links as expected, but you need to run a Editing or adding files works fine. @Krzysztof-Cieslak Is there any other testing or documentation you would expect? |
@baronfel This fixes a known bug. Do you see any issues that this shouldn't be put into a bugfix release? |
Thanks for implementing @bigjonrobers, and for testing @rdipardo! I'll merge and make a bugfix release. |
This resolves #96 by only initializing SiteContents once when in watch mode.
I also removed an option for config in Generator.fs, as we're already potentially raising another exception in that let binding, so loading out and option and then immediately raising an exception on
None
didn't seem to add any value but to clutter up the code.