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