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

advanced/folder-uselargeblocks: Defaults for "useLargeBlocks" have changed #454

Merged
merged 2 commits into from Jun 3, 2019

Conversation

Projects
None yet
2 participants
@GermanCoding
Copy link
Contributor

commented Jun 3, 2019

I believe there is no issue for this PR (on the tracker), but since this is a really small change and calmh suggested a PR, I just created this.

Related to this forum discussion: https://forum.syncthing.net/t/variable-block-size-support-on-by-default/13334/

This change updates the documentation and reflects the fact that all Syncthing versions since 1.1.0 have useLargeBlocks enabled by default (for new folders).

Additionally, there was an issue with my sphinx nagging me that a lexer called cmd does not exist (dev/building.rst is the only file declaring this). I fixed this by changing it to batch which is recommended by this page. This changes the layout of the code section slightly, it adds a minor amount of syntax highlighting to the Windows section which was previously missing.

@AudriusButkevicius AudriusButkevicius merged commit 87f198e into syncthing:master Jun 3, 2019

1 check passed

Build Documentation (Web) TeamCity build finished
Details

@GermanCoding GermanCoding deleted the GermanCoding:doc_updates branch Jun 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.