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 logging documentation #6904

Merged
merged 4 commits into from May 11, 2019

Conversation

5 participants
@zeripath
Copy link
Contributor

commented May 10, 2019

The current logging documentation says use ENABLE_MACARON_REDIRECT by mistake instead of REDIRECT_MACARON_LOG

It's also somewhat awkward to have to set DISABLE_ROUTER_LOG in [server] section instead of the [log] section.

This PR fixes both these issues.

zeripath added some commits May 10, 2019

@codecov-io

This comment has been minimized.

Copy link

commented May 10, 2019

Codecov Report

Merging #6904 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6904      +/-   ##
==========================================
+ Coverage    41.4%   41.42%   +0.02%     
==========================================
  Files         432      432              
  Lines       59541    59543       +2     
==========================================
+ Hits        24655    24668      +13     
+ Misses      31647    31635      -12     
- Partials     3239     3240       +1
Impacted Files Coverage Δ
modules/setting/log.go 73.73% <100%> (+0.24%) ⬆️
routers/repo/view.go 43.03% <0%> (+1.01%) ⬆️
models/repo_list.go 67.89% <0%> (+1.05%) ⬆️
modules/log/event.go 65.98% <0%> (+1.52%) ⬆️
models/unit.go 67.56% <0%> (+5.4%) ⬆️

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 f378cb5...42c1d3e. Read the comment docs.

@techknowlogick techknowlogick added this to the 1.9.0 milestone May 11, 2019

@GiteaBot GiteaBot added lgtm/need 1 and removed lgtm/need 2 labels May 11, 2019

@GiteaBot GiteaBot added lgtm/done and removed lgtm/need 1 labels May 11, 2019

@zeripath zeripath merged commit e316674 into go-gitea:master May 11, 2019

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr Build is passing
Details

zeripath added a commit to zeripath/gitea that referenced this pull request May 11, 2019

Fix logging documentation (go-gitea#6904)
* ENABLE_MACARON_REDIRECT should be REDIRECT_MACARON_LOG

* Allow DISABLE_ROUTER_LOG to be set in the [log] section

techknowlogick added a commit that referenced this pull request May 11, 2019

Remove local clones & make hooks run on merge/edit/upload (#6672)
* Add options to git.Clone to make it more capable

* Begin the process of removing the local copy and tidy up

* Remove Wiki LocalCopy Checkouts

* Remove the last LocalRepo helpers

* Remove WithTemporaryFile

* Enable push-hooks for these routes

* Ensure tests cope with hooks

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Remove Repository.LocalCopyPath()

* Move temporary repo to use the standard temporary path

* Fix the tests

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Remove LocalWikiPath

* Fix missing remove

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Use AppURL for Oauth user link (#6894)

* Use AppURL for Oauth user link

Fix #6843

* Update oauth.go

* Update oauth.go

* internal/ssh: ignore env command totally (#6825)

* ssh: ignore env command totally

* Remove commented code 

Needed fix described in issue #6889

* Escape the commit message on issues update and title in telegram hook (#6901)

* update sdk to latest (#6903)

* improve description of branch protection (fix #6886) (#6906)

The branch protection description text were not quite accurate.

* Fix logging documentation (#6904)

* ENABLE_MACARON_REDIRECT should be REDIRECT_MACARON_LOG

* Allow DISABLE_ROUTER_LOG to be set in the [log] section

* [skip ci] Updated translations via Crowdin

* Move sdk structs to modules/structs (#6905)

* move sdk structs to moduels/structs

* fix tests

* fix fmt

* fix swagger

* fix vendor

@zeripath zeripath deleted the zeripath:fix-logging-documentation branch May 20, 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.