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

Clean up docs #5274

Merged
merged 7 commits into from Nov 5, 2018

Conversation

7 participants
@techknowlogick
Member

techknowlogick commented Nov 4, 2018

  • Add documentation on how to add analytics
  • Make sure to say the version that themes were added in
  • Fix #5074

techknowlogick added some commits Nov 4, 2018

@codecov-io

This comment has been minimized.

codecov-io commented Nov 4, 2018

Codecov Report

Merging #5274 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #5274      +/-   ##
=========================================
- Coverage   37.63%   37.6%   -0.03%     
=========================================
  Files         310     310              
  Lines       46054   46054              
=========================================
- Hits        17331   17320      -11     
- Misses      26246   26254       +8     
- Partials     2477    2480       +3
Impacted Files Coverage Δ
models/repo_indexer.go 44.49% <0%> (-3.82%) ⬇️
models/repo_list.go 63.29% <0%> (-1.27%) ⬇️

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 0fd56a7...6726f14. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 label Nov 4, 2018

@sapk

sapk approved these changes Nov 4, 2018

Show resolved Hide resolved custom/conf/app.ini.sample Outdated
Show resolved Hide resolved custom/conf/app.ini.sample Outdated

techknowlogick added some commits Nov 4, 2018

@lunny

lunny approved these changes Nov 5, 2018

@bkcsoft bkcsoft added lgtm/done and removed lgtm/need 1 labels Nov 5, 2018

@techknowlogick techknowlogick merged commit 92fb89f into go-gitea:master Nov 5, 2018

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr the build was successful
Details

@techknowlogick techknowlogick deleted the techknowlogick:techknowlogick-patch-9 branch Nov 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment