github / github/gh-classroom

Linting fails after forking and cloning

Aperta
#96 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Go
Stelle
125
Fork
21
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

I was going to fix the error in CONTRIBUTING.md I mentioned in Issue #95. The `golangci-lint run` command results in the following...

> gh-classroom on  main via 🐹 v1.23.6 took 18s
❯ `golangci-lint run`
pkg/classroom/classroom.go:144:21: printf: non-constant format string in call to fmt.Sprintf (govet)
return fmt.Sprintf(a.Classroom.Url)

## Context
### Platform
MacOS
Also replicated it on CachyOS Linux (Arch based distro)
### Submitting a pull request
1. [Fork](https://github.com/github/gh-classroom/fork) and clone the repository
2. Configure and install the dependencies: `go mod tidy`
3. Make sure the tests pass on your machine: `go test -v ./...`
4. Make sure linter passes on your machine: `golangci-lint run` <----- This line causes the error
5. Create a new branch: `git checkout -b my-branch-name`
6. Make your change, add tests, and make sure the tests and linter still pass
7. Push to your fork and [submit a pull request](https://github.com/github/gh-classroom/compare)
8. Please be patient while waiting for your pull request to be reviewed and merged.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da pkg/classroom/classroom.go:144, dove si verifica l’errore govet segnalato nella chiamata a fmt.Sprintf. Esegui golangci-lint run e go test -v ./... prima e dopo la modifica; il lavoro è completato quando l’errore di lint è risolto e la suite di test continua a passare.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
go
Ambito
cli
Tipo di issue
Bug
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
58/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.