Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] revive: update to 1.3.2.
@ 2023-05-21 19:10 r-ricci
  2023-05-21 23:54 ` [PR PATCH] [Merged]: " abenson
  0 siblings, 1 reply; 3+ messages in thread
From: r-ricci @ 2023-05-21 19:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1235 bytes --]

There is a new pull request by r-ricci against master on the void-packages repository

https://github.com/r-ricci/void-packages revive
https://github.com/void-linux/void-packages/pull/44019

revive: update to 1.3.2.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/44019.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-revive-44019.patch --]
[-- Type: text/x-diff, Size: 3756 bytes --]

From eda7c53250f05a5ee05060c4d16f22d9ae2dd14b Mon Sep 17 00:00:00 2001
From: Roberto Ricci <ricci@disroot.org>
Date: Sun, 21 May 2023 20:54:41 +0200
Subject: [PATCH] revive: update to 1.3.2.

---
 srcpkgs/revive/patches/fix-tests.patch | 53 --------------------------
 srcpkgs/revive/template                |  4 +-
 2 files changed, 2 insertions(+), 55 deletions(-)
 delete mode 100644 srcpkgs/revive/patches/fix-tests.patch

diff --git a/srcpkgs/revive/patches/fix-tests.patch b/srcpkgs/revive/patches/fix-tests.patch
deleted file mode 100644
index 2e35877f95c3..000000000000
--- a/srcpkgs/revive/patches/fix-tests.patch
+++ /dev/null
@@ -1,53 +0,0 @@
-From dc6909b49ff16af43ac29ce1e61265be33a3bf28 Mon Sep 17 00:00:00 2001
-From: chavacava <salvadorcavadini+github@gmail.com>
-Date: Mon, 1 May 2023 10:30:48 +0200
-Subject: [PATCH] fix tests not passing after the modif of the default rules
- list (#817)
-
----
- revivelib/core_test.go | 17 +++++++----------
- 1 file changed, 7 insertions(+), 10 deletions(-)
-
-diff --git a/revivelib/core_test.go b/revivelib/core_test.go
-index aa0ddb48..44ab6276 100644
---- a/revivelib/core_test.go
-+++ b/revivelib/core_test.go
-@@ -26,17 +26,12 @@ func TestReviveLint(t *testing.T) {
- 		failureList = append(failureList, failure)
- 	}
- 
--	const expected = 3
-+	const expected = 5
- 
- 	got := len(failureList)
- 	if got != expected {
- 		t.Fatalf("Expected failures to have %d failures, but it has %d.", expected, got)
- 	}
--
--	errmsg := "redundant if ...; err != nil check, just return error instead."
--	if failureList[0].Failure != errmsg {
--		t.Fatalf("Expected failure[0] to be '%s', but it was '%s'", errmsg, failureList[0].Failure)
--	}
- }
- 
- func TestReviveFormat(t *testing.T) {
-@@ -57,13 +52,15 @@ func TestReviveFormat(t *testing.T) {
- 	}
- 
- 	errorMsgs := []string{
--		"(15, 2)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
--		"(88, 3)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
--		"(95, 3)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
-+		"(91, 3)  https://revive.run/r#unreachable-code  unreachable code after this statement",
-+		"(98, 3)  https://revive.run/r#unreachable-code  unreachable code after this statement",
-+		"(15, 2)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
-+		"(88, 3)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
-+		"(95, 3)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
- 	}
- 	for _, errorMsg := range errorMsgs {
- 		if !strings.Contains(failures, errorMsg) {
--			t.Fatalf("Expected formatted failures '%s' to contain '%s', but it didn't.", failures, errorMsg)
-+			t.Fatalf("Expected formatted failures\n'%s'\nto contain\n'%s', but it didn't.", failures, errorMsg)
- 		}
- 	}
- 
diff --git a/srcpkgs/revive/template b/srcpkgs/revive/template
index 54ebb05633d8..c6958416ddbc 100644
--- a/srcpkgs/revive/template
+++ b/srcpkgs/revive/template
@@ -1,6 +1,6 @@
 # Template file for 'revive'
 pkgname=revive
-version=1.3.1
+version=1.3.2
 revision=1
 build_style=go
 go_ldflags="-X github.com/mgechev/revive/cli.version=$version"
@@ -11,7 +11,7 @@ license="MIT"
 homepage="https://revive.run/"
 changelog="https://github.com/mgechev/revive/releases"
 distfiles="https://github.com/mgechev/revive/archive/refs/tags/v${version}.tar.gz"
-checksum=b3b8804024cd2aadb0900912560ad8718d5d1fc47fba7d95d281b46a24352267
+checksum=3abccf0a2f2d9f257afbcfb48d96668ebb5fb360d2b460f9c1f166a9711ddeaa
 
 do_check() {
 	go test -v ./...

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PR PATCH] [Merged]: revive: update to 1.3.2.
  2023-05-21 19:10 [PR PATCH] revive: update to 1.3.2 r-ricci
@ 2023-05-21 23:54 ` abenson
  0 siblings, 0 replies; 3+ messages in thread
From: abenson @ 2023-05-21 23:54 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1085 bytes --]

There's a merged pull request on the void-packages repository

revive: update to 1.3.2.
https://github.com/void-linux/void-packages/pull/44019

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PR PATCH] revive: update to 1.3.2.
@ 2023-05-20 19:03 icp1994
  0 siblings, 0 replies; 3+ messages in thread
From: icp1994 @ 2023-05-20 19:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 474 bytes --]

There is a new pull request by icp1994 against master on the void-packages repository

https://github.com/icp1994/void-packages revive
https://github.com/void-linux/void-packages/pull/43972

revive: update to 1.3.2.
#### Testing the changes
- I tested the changes in this PR: **briefly**

#### Local build testing
- I built this PR locally for my native architecture: **x86_64**

A patch file from https://github.com/void-linux/void-packages/pull/43972.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-revive-43972.patch --]
[-- Type: text/x-diff, Size: 3749 bytes --]

From 2555e0acb10252cbce7c6ec75bc8206920ddfc69 Mon Sep 17 00:00:00 2001
From: icp <pangolin@vivaldi.net>
Date: Fri, 19 May 2023 15:37:41 +0530
Subject: [PATCH] revive: update to 1.3.2.

---
 srcpkgs/revive/patches/fix-tests.patch | 53 --------------------------
 srcpkgs/revive/template                |  4 +-
 2 files changed, 2 insertions(+), 55 deletions(-)
 delete mode 100644 srcpkgs/revive/patches/fix-tests.patch

diff --git a/srcpkgs/revive/patches/fix-tests.patch b/srcpkgs/revive/patches/fix-tests.patch
deleted file mode 100644
index 2e35877f95c3..000000000000
--- a/srcpkgs/revive/patches/fix-tests.patch
+++ /dev/null
@@ -1,53 +0,0 @@
-From dc6909b49ff16af43ac29ce1e61265be33a3bf28 Mon Sep 17 00:00:00 2001
-From: chavacava <salvadorcavadini+github@gmail.com>
-Date: Mon, 1 May 2023 10:30:48 +0200
-Subject: [PATCH] fix tests not passing after the modif of the default rules
- list (#817)
-
----
- revivelib/core_test.go | 17 +++++++----------
- 1 file changed, 7 insertions(+), 10 deletions(-)
-
-diff --git a/revivelib/core_test.go b/revivelib/core_test.go
-index aa0ddb48..44ab6276 100644
---- a/revivelib/core_test.go
-+++ b/revivelib/core_test.go
-@@ -26,17 +26,12 @@ func TestReviveLint(t *testing.T) {
- 		failureList = append(failureList, failure)
- 	}
- 
--	const expected = 3
-+	const expected = 5
- 
- 	got := len(failureList)
- 	if got != expected {
- 		t.Fatalf("Expected failures to have %d failures, but it has %d.", expected, got)
- 	}
--
--	errmsg := "redundant if ...; err != nil check, just return error instead."
--	if failureList[0].Failure != errmsg {
--		t.Fatalf("Expected failure[0] to be '%s', but it was '%s'", errmsg, failureList[0].Failure)
--	}
- }
- 
- func TestReviveFormat(t *testing.T) {
-@@ -57,13 +52,15 @@ func TestReviveFormat(t *testing.T) {
- 	}
- 
- 	errorMsgs := []string{
--		"(15, 2)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
--		"(88, 3)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
--		"(95, 3)  https://revive.run/r#if-return  redundant if ...; err != nil check, just return error instead.",
-+		"(91, 3)  https://revive.run/r#unreachable-code  unreachable code after this statement",
-+		"(98, 3)  https://revive.run/r#unreachable-code  unreachable code after this statement",
-+		"(15, 2)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
-+		"(88, 3)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
-+		"(95, 3)  https://revive.run/r#if-return         redundant if ...; err != nil check, just return error instead.",
- 	}
- 	for _, errorMsg := range errorMsgs {
- 		if !strings.Contains(failures, errorMsg) {
--			t.Fatalf("Expected formatted failures '%s' to contain '%s', but it didn't.", failures, errorMsg)
-+			t.Fatalf("Expected formatted failures\n'%s'\nto contain\n'%s', but it didn't.", failures, errorMsg)
- 		}
- 	}
- 
diff --git a/srcpkgs/revive/template b/srcpkgs/revive/template
index 54ebb05633d8..c6958416ddbc 100644
--- a/srcpkgs/revive/template
+++ b/srcpkgs/revive/template
@@ -1,6 +1,6 @@
 # Template file for 'revive'
 pkgname=revive
-version=1.3.1
+version=1.3.2
 revision=1
 build_style=go
 go_ldflags="-X github.com/mgechev/revive/cli.version=$version"
@@ -11,7 +11,7 @@ license="MIT"
 homepage="https://revive.run/"
 changelog="https://github.com/mgechev/revive/releases"
 distfiles="https://github.com/mgechev/revive/archive/refs/tags/v${version}.tar.gz"
-checksum=b3b8804024cd2aadb0900912560ad8718d5d1fc47fba7d95d281b46a24352267
+checksum=3abccf0a2f2d9f257afbcfb48d96668ebb5fb360d2b460f9c1f166a9711ddeaa
 
 do_check() {
 	go test -v ./...

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-05-21 23:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-21 19:10 [PR PATCH] revive: update to 1.3.2 r-ricci
2023-05-21 23:54 ` [PR PATCH] [Merged]: " abenson
  -- strict thread matches above, loose matches on Subject: below --
2023-05-20 19:03 [PR PATCH] " icp1994

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).