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

Improvements and more begginer-friendly README #1709

Closed
wants to merge 1 commit into from
Closed

Improvements and more begginer-friendly README #1709

wants to merge 1 commit into from

Conversation

samueldcs
Copy link

I have updated the file to make it easier for beginners to get started, and also fixed some weird things.

I have updated the file to make it easier for beginners to get started, and also fixed some weird things.
@samueldcs samueldcs changed the title Improvements and more begginer-friendly. Improvements and more begginer-friendly README Dec 20, 2018
@@ -81,14 +83,13 @@ $ go get -u github.com/gin-gonic/gin
```go
import "github.com/gin-gonic/gin"
```

3. (Optional) Import `net/http`. This is required for example if using constants such as `http.StatusOK`.
3. (Optional) Import `net/http`. This is required if you want to use constants such as `http.StatusOK`.
Copy link
Member

Choose a reason for hiding this comment

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

add an empty line before "```".

@@ -2035,8 +2042,7 @@ func TestPingRoute(t *testing.T) {
```

## Users

Awesome project lists using [Gin](https://github.com/gin-gonic/gin) web framework.
List of awesome projects made with the [Gin](https://github.com/gin-gonic/gin) web framework.
Copy link
Member

Choose a reason for hiding this comment

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

Add an empty line after ## Users

@appleboy
Copy link
Member

and please also check errors from Travis

@codecov
Copy link

codecov bot commented Dec 26, 2018

Codecov Report

Merging #1709 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1709      +/-   ##
==========================================
+ Coverage    99.4%   99.42%   +0.01%     
==========================================
  Files          42       42              
  Lines        2029     2073      +44     
==========================================
+ Hits         2017     2061      +44     
  Misses          9        9              
  Partials        3        3
Impacted Files Coverage Δ
logger.go 100% <0%> (ø) ⬆️

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 678e09c...52f383c. Read the comment docs.

@thinkerou thinkerou added this to the 1.4 milestone Feb 26, 2019
@thinkerou thinkerou self-requested a review March 1, 2019 02:27
Copy link
Member

@thinkerou thinkerou left a comment

Choose a reason for hiding this comment

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

@samueldcs please fix comment and I will merge it, thanks!

@thinkerou thinkerou modified the milestones: 1.4, 1.5 Mar 1, 2019
@thinkerou
Copy link
Member

@samueldcs thanks! closing, please see #1793

@thinkerou thinkerou closed this Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants