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 AUTOTEMP documentation in configs #2868

Merged
merged 20 commits into from Feb 17, 2016
Merged

Fix AUTOTEMP documentation in configs #2868

merged 20 commits into from Feb 17, 2016

Conversation

ghost
Copy link

@ghost ghost commented Dec 22, 2015

No description provided.

@Wackerbarth
Copy link
Contributor

In the Marlin 1.1 series, this duplication was intentional because of the way that you were instructed to perform configuration. In particular, if any part of the configuration file does not match the "default" version, the entire file is duplicated.

A different methodology is being pursued in the Marlin 1.2 series now being developed in the MarlinDev repository.

@thinkyhead
Copy link
Member

Looks like the title of this should really be "Fix AUTOTEMP documentation in configs," since it only moves the AUTOTEMP comments to the line above AUTOTEMP where they should have been in the first place. As such, I think it's a helpful tweak for 1.1.0.

@thinkyhead thinkyhead changed the title duplicated documents. Fix AUTOTEMP documentation in configs Jan 27, 2016
thinkyhead added a commit that referenced this pull request Feb 17, 2016
Fix AUTOTEMP documentation in configs
@thinkyhead thinkyhead merged commit b037872 into MarlinFirmware:RCBugFix Feb 17, 2016
@ghost ghost deleted the esenapaj-patch-1 branch February 18, 2016 19:10
@thinkyhead thinkyhead mentioned this pull request Mar 19, 2016
@jbrazio jbrazio modified the milestone: 1.1.0 Jul 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants