8000 chore(main): release 3.1.0 by theoludwig · Pull Request #402 · editorconfig-checker/editorconfig-checker · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore(main): release 3.1.0 #402

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

Closed
wants to merge 1 commit into from
Closed

Conversation

theoludwig
Copy link
Member
@theoludwig theoludwig commented Jan 5, 2025

🤖 I have created a release beep boop

3.1.0 (2025-01-05)

Features

  • add zip version when compressing all binaries (#321) (#362) (f1bb625)
  • consolidate adjacent error messages (#360) (cf4ae1c)
  • editorconfig-checker-disable-next-line (#363) (6116ec6)
  • provide Codeclimate compatible report fromat (#367) (282c315)
  • support .editorconfig-checker.json config (#375) (cb0039c)

Bug Fixes


This PR was generated with Release Please. See documentation.

@theoludwig
Copy link
Member Author

@klaernie I've used my GitHub token, instead of the one already defined, as it was not working, "Bad credentials" (ref: https://github.com/editorconfig-checker/editorconfig-checker/actions/runs/12623086839/job/35171698632).
It's probably fine like that, but maybe we can directly use GITHUB_TOKEN instead of our own defined RELEASE_PLEASE_TOKEN, that way, I believe, it is the GitHub Actions bot, instead of my account creating the PRs and Releases?

Also, the CI is failing, it seems, that it's replacing the LineNumber in the tests.

@@ -28,15 +28,15 @@ func TestGetErrorCount(t *testing.T) {
FilePath: "some/path",
Errors: []ValidationError{
{
LineNumber: 1,
LineNumber: 3,
Copy link
Member Author

Choose a reason for hiding this comment

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

It should not replace LineNumber by 3 here.

Do you have an idea, why the bot does that? @klaernie

@@ -19,7 +19,7 @@ import (

func TestGetErrorCount(t *testing.T) {
count := GetErrorCount([]ValidationErrors{})
if count != 0 {
if count != 3 {
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't get this change, nor anything that come next (about line and count)

I mean how is it related to bumping version 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I agree. We should check what went wrong with the Release Please bot.

Copy link
Member

Choose a reason for hiding this comment

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

Oh shit - I forgot to replace the start by stop in line 15

@theoludwig theoludwig force-pushed the release-please--branches--main branch from 8fe414f to 4b60598 Compare January 5, 2025 22:26
@klaernie
Copy link
Member
klaernie commented Jan 5, 2025

Let's close this one, and rerun the action on top of the last commit, IMHO that should create a new PR which does not attempt to replace every number

@klaernie klaernie closed this Jan 5, 2025
@klaernie klaernie deleted the release-please--branches--main branch January 5, 2025 23:17
@theoludwig theoludwig removed this from the v3.1.0 milestone Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0