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

[2.7] Add docs for autoredirect config parameter #2804

Merged
merged 1 commit into from
Jan 7, 2019

Conversation

caervs
Copy link
Contributor

@caervs caervs commented Jan 7, 2019

Config parameter is user facing so should be documented.

Signed-off-by: Ryan Abrams rdabrams@gmail.com

Config parameter is user facing so should be documented.

Signed-off-by: Ryan Abrams <rdabrams@gmail.com>
@caervs
Copy link
Contributor Author

caervs commented Jan 7, 2019

Port #2801 to 2.7 branch

Depends on #2800 and addresses #2793

@codecov
Copy link

codecov bot commented Jan 7, 2019

Codecov Report

Merging #2804 into release/2.7 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/2.7    #2804   +/-   ##
============================================
  Coverage        60.24%   60.24%           
============================================
  Files              103      103           
  Lines             8022     8022           
============================================
  Hits              4833     4833           
  Misses            2546     2546           
  Partials           643      643
Flag Coverage Δ
#linux 60.24% <ø> (ø) ⬆️

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 40b7b58...d1abdeb. Read the comment docs.

Copy link
Collaborator

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@caervs caervs requested a review from davidswu January 7, 2019 22:23
Copy link
Contributor

@davidswu davidswu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@caervs caervs merged commit 62994fd into distribution:release/2.7 Jan 7, 2019
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.

None yet

3 participants