Conversation
Update README
Models and Modules
Setup golangci-lint
Create initial card and deck models
pythoninthegrass
force-pushed
the
dev
branch
from
ae4b112 to
42c0828
Compare
Contributor
Author
|
... only to add further clarification to pre-commit hooks afterward 😅 |
pythoninthegrass
force-pushed
the
dev
branch
5 times, most recently
from
525ec91 to
1e3f73f
Compare
- `README.md`: update toc, remove line breaks (personal prefs) - `.editorconfig`: opinionated code style configuration - `.markdownlint.jsonc`: markdown linter configuration - `.pre-commit-config.yaml`: pre-commit hooks (e.g., `golangci-lint`) - `.tool-versions`: asdf for runtimes - `.vscode/extensions.json`: automatically install extensions in vscode - `.vscode/launch.json`: basic debug config for go - `.vscode/settings.json`: workspace settings to handle go source code
pythoninthegrass
force-pushed
the
dev
branch
from
1e3f73f to
d3e58d4
Compare
There was a problem hiding this comment.
PR Overview
This PR adds tooling configurations to improve the development workflow and enforce code quality checks for the Go project.
- Adds a pre-commit configuration for linting, security, and formatting using various repositories.
- Updates the README.md to enhance the Table of Contents and add minimum/recommended development requirements.
Reviewed Changes
| File | Description |
|---|---|
| .pre-commit-config.yaml | Introduces multiple pre-commit hooks to enforce code quality and security checks. |
| README.md | Updates the Table of Contents and development requirements sections for clarity. |
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi Cal!
Made some changes to help out your tooling ecosystem to hopefully supercharge your go journey.
Feel free to take what you want and discard the rest 👌
Summary of changes
README.md: update toc, add minimum/rec'd dev reqs, remove line breaks (personal prefs).editorconfig: opinionated code style configuration.markdownlint.jsonc: markdown linter configuration.pre-commit-config.yaml: pre-commit hooks (e.g.,golangci-lint).tool-versions: asdf for runtimes.vscode/extensions.json: automatically install extensions in vscode.vscode/launch.json: basic debug config for go.vscode/settings.json: workspace settings to handle go source code