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

Add EditorConfig #69

Closed
wants to merge 411 commits into from

Conversation

hyperupcall
Copy link

As alluded to in this PR, this adds an EditorConfig. It's similar to that in the original PR, except the default indent_style was changed to tab, since the majority of the code is written in Go. The rest of the file is written as a result.

FYI: The link in .github/CONTRIBUTING.md is broken

dvaumoron and others added 24 commits February 7, 2024 13:31
Bumps [github.com/ProtonMail/gopenpgp/v2](https://github.com/ProtonMail/gopenpgp) from 2.7.4 to 2.7.5.
- [Release notes](https://github.com/ProtonMail/gopenpgp/releases)
- [Changelog](https://github.com/ProtonMail/gopenpgp/blob/main/CHANGELOG.md)
- [Commits](ProtonMail/gopenpgp@v2.7.4...v2.7.5)

---
updated-dependencies:
- dependency-name: github.com/ProtonMail/gopenpgp/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 3 to 4.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v3...v4)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…ons/golangci/golangci-lint-action-4

ci: bump golangci/golangci-lint-action from 3 to 4
Add docker container build and push workflow
…github.com/ProtonMail/gopenpgp/v2-2.7.5

ci: bump github.com/ProtonMail/gopenpgp/v2 from 2.7.4 to 2.7.5
Signed-off-by: Edwin Kofler <edwin@kofler.dev>
@dvaumoron
Copy link
Contributor

Go has its own formatting rules, so maybe *.go file should be excluded

@kvendingoldo
Copy link
Collaborator

agree about go

@hyperupcall
Copy link
Author

hyperupcall commented Feb 18, 2024

Are there any specific rules that you would like to be excluded? As long as the default Go formatting rules don't contradict the EditorConfig, then contributors will still automatically get the correct file editing settings. If Go is excluded as a whole, then settings may be incorrect when typing in a Go file, and only fixed if go fmt is ran. Here is an example of an EditorConfig in kubernetes/dashboard that seems to work well.

@dvaumoron
Copy link
Contributor

That's not my point, an editor supporting Go should already enforce Go formatting rules with go fmt on save, so a second check does not seem useful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants