Skip to content

Commit

Permalink
Update linter, fix its errors (#32)
Browse files Browse the repository at this point in the history
* update linter version

* CI: update checkout version as well

* CI: depth

* Fix linter issues of linter itself

* lint readme

* add .perlcriticrc

* maybe fix lint in readme

* try again

* mention this in changes
  • Loading branch information
DarthGandalf authored Oct 12, 2024
1 parent c52012a commit b0cdf71
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 13 deletions.
9 changes: 6 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,13 @@ name: Test
on:
- push
- pull_request
permissions: {}
jobs:
test:
name: Test
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
# Some deps can't be installed purely from CPAN. And some other are just slow to install from there.
- run: sudo apt install libimage-magick-perl libgtk3-perl libglib-object-introspection-perl libtest-differences-perl libcarp-always-perl libdist-zilla-perl cpanminus xvfb
# Install everything which is still not installed, but don't mess current directory with root-owned files
Expand All @@ -26,8 +27,10 @@ jobs:
name: Lint
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: github/super-linter@v3
- uses: actions/checkout@v4
with:
fetch-depth: 0
- uses: super-linter/[email protected]
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
FILTER_REGEX_EXCLUDE: Makefile.PL
1 change: 1 addition & 0 deletions .perlcriticrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
exclude = ProhibitBitwiseOperators
1 change: 1 addition & 0 deletions Changes
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
{{$NEXT}}
- Removed perl feature switch, for compatibility with newer perl
- Made linter on CI happy

11 2024-10-01 17:08:47 IST
- Replaced deprecated given/when with if/elsif/else
Expand Down
20 changes: 10 additions & 10 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ Gtk3 imager viewer widget modelled after the GtkImageView C widget by Björn
Lindqvist <[email protected]>

To discuss Gtk3::ImageView or gtk3-perl, ask questions and flame/praise the
authors, join [email protected] at lists.gnome.org.
authors, join [mailing list](mailto:[email protected]) at lists.gnome.org.

## INSTALLATION

Expand All @@ -28,24 +28,24 @@ Or you can use `dzil build` to get the directory of the release in current dir,
`dzil help`, [dzil.org](http://dzil.org) and [Dist::Zilla::Tutorial](https://metacpan.org/pod/Dist::Zilla::Tutorial#BUILDING-YOUR-DIST) have more details.

This will install the module to the subdirectory lib/perl5 under the given
prefix. If this is not already in perl's include path, you'll need to tell
prefix. If this is not already in perl's include path, you'll need to tell
perl how to get to this library directory so you can use it; there are three
ways:

* in your environment (the easiest):
- in your environment (the easiest):

```shell
PERL5LIB=/some/other/place/lib/perl5/site_perl
export PERL5LIB
```

* on the perl command line:
- on the perl command line:

```shell
perl -I /some/other/place/lib/perl5/site_perl yourscript
```

* in the code of your perl script:
- in the code of your perl script:

```shell
use lib '/some/other/place/lib/perl5/site_perl';
Expand All @@ -55,11 +55,11 @@ use lib '/some/other/place/lib/perl5/site_perl';

This module requires these other modules and libraries:

* perl >= 5.8.0
* Glib >= 1.163 (Perl module)
* GTK+ 3.x (C library)
* Gtk3 (Perl module)
* Readonly (Perl module)
- perl >= 5.8.0
- Glib >= 1.163 (Perl module)
- GTK+ 3.x (C library)
- Gtk3 (Perl module)
- Readonly (Perl module)

## BUG REPORTS

Expand Down

0 comments on commit b0cdf71

Please sign in to comment.