Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.

Basic message about lack of maintenance#77

Merged
pbrisbin merged 5 commits intopbrisbin:masterfrom
glenntanner3:defunct
Feb 3, 2021
Merged

Basic message about lack of maintenance#77
pbrisbin merged 5 commits intopbrisbin:masterfrom
glenntanner3:defunct

Conversation

@glenntanner3
Copy link
Contributor

To the point message with a 1 second sleep so the users are aware

Copy link
Owner

@pbrisbin pbrisbin left a comment

Choose a reason for hiding this comment

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

Thank you!

I will merge and release this week or next at the latest.

I have spent too much time on gitlab for work.....

Co-authored-by: patrick brisbin <pbrisbin@gmail.com>
@pbrisbin
Copy link
Owner

pbrisbin commented Feb 3, 2021

Would you mind getting the tests green? The test suite asserts against expected output, so the new warning line is causing these failures. It should just require adding that line to the expected output in the tests. You can use a glob to avoid asserting the exact text.

For example, in build.t:

   $ source "$TESTDIR/helper.sh"
 
 Building the package for aurget
 
   $ aurget -Sb --builddir "$PWD" --noedit --nodeps --noconfirm --nodiscard aurget
+  warning: * (glob)
   :: Searching AUR...
   
   Targets (1): aurget-4.6.0-1 

This'll need to be done in all the assertions of all the failing specs.

@pbrisbin pbrisbin merged commit 05a7795 into pbrisbin:master Feb 3, 2021
@glenntanner3 glenntanner3 deleted the defunct branch February 3, 2021 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants