Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] revive: update to 1.3.1.
@ 2023-05-16 18:49 icp1994
  2023-05-17  4:50 ` [PR PATCH] [Merged]: " classabbyamp
  0 siblings, 1 reply; 2+ messages in thread
From: icp1994 @ 2023-05-16 18:49 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 470 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/43905

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

#### 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/43905.patch is attached

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

From 5940d37fcf207899e547b24d3c91bc20e7f8e1f1 Mon Sep 17 00:00:00 2001
From: icp <pangolin@vivaldi.net>
Date: Thu, 16 Mar 2023 01:08:34 +0530
Subject: [PATCH] revive: update to 1.3.1.

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

diff --git a/srcpkgs/revive/patches/fix-tests.patch b/srcpkgs/revive/patches/fix-tests.patch
new file mode 100644
index 000000000000..2e35877f95c3
--- /dev/null
+++ b/srcpkgs/revive/patches/fix-tests.patch
@@ -0,0 +1,53 @@
+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 0828ab00d6f1..54ebb05633d8 100644
--- a/srcpkgs/revive/template
+++ b/srcpkgs/revive/template
@@ -1,7 +1,7 @@
 # Template file for 'revive'
 pkgname=revive
-version=1.2.4
-revision=2
+version=1.3.1
+revision=1
 build_style=go
 go_ldflags="-X github.com/mgechev/revive/cli.version=$version"
 go_import_path="github.com/mgechev/revive"
@@ -10,8 +10,8 @@ maintainer="Roberto Ricci <ricci@disroot.org>"
 license="MIT"
 homepage="https://revive.run/"
 changelog="https://github.com/mgechev/revive/releases"
-distfiles="https://github.com/mgechev/revive/archive/v$version.tar.gz"
-checksum=bf2439a1a487274b7e37f84394dab87c2418e5396baffd501eb8543889564a2c
+distfiles="https://github.com/mgechev/revive/archive/refs/tags/v${version}.tar.gz"
+checksum=b3b8804024cd2aadb0900912560ad8718d5d1fc47fba7d95d281b46a24352267
 
 do_check() {
 	go test -v ./...

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

* Re: [PR PATCH] [Merged]: revive: update to 1.3.1.
  2023-05-16 18:49 [PR PATCH] revive: update to 1.3.1 icp1994
@ 2023-05-17  4:50 ` classabbyamp
  0 siblings, 0 replies; 2+ messages in thread
From: classabbyamp @ 2023-05-17  4:50 UTC (permalink / raw)
  To: ml

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

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

revive: update to 1.3.1.
https://github.com/void-linux/void-packages/pull/43905

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

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

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

end of thread, other threads:[~2023-05-17  4:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-16 18:49 [PR PATCH] revive: update to 1.3.1 icp1994
2023-05-17  4:50 ` [PR PATCH] [Merged]: " classabbyamp

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).