Since Go 1.20,

> [t]he fmt.Errorf function now supports multiple occurrences of the %w
> format verb, which will cause it to return an error that wraps all of
> those error operands.

This change, together with newer errorlint, causes the following
warning:

> mount_unix.go:74:56: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
> 					return fmt.Errorf("%w (possible cause: %s)", err, suberr)
>					                                                  ^

As we still want to support Go 1.17, use explicit string for suberr,
and add a TODO to switch to %w once we switch to Go 1.20+.

Fixes: 487129d
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

1. Test with supported Go releases (1.20.x and 1.21.x). Also leave Go
   1.17.x since it costs us little to keep supporting it.

2. Bump golangci-lint to the latest released version (v1.55.1).

3. Bump actions/setup-go to v4.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

It is already that way for sequential, and Go 1.17 is the oldest version
we use in CI, so let's bring other modules go.mod in sync.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

This is the oldest tagged version of x/sys. This allows to follow Go
module's "minimum version selection", leaving it up to users of this
module to pick the version they want.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

golangci-lint v1.54.2 comes with errorlint v1.4.4, which contains
the fix [1] whitelisting all errno comparisons for errors coming from
x/sys/unix. Remove the annotation that is no longer needed.

Unfortunately, switch on a bare unix error (in mountinfo) still needs to
be annotated (see [2]).

[1] polyfloyd/go-errorlint#47
[2] polyfloyd/go-errorlint#54

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

This fixes the following CI error:

> sequential_windows.go:114: File is not `gofumpt`-ed (gofumpt)

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

Bump CI to use supported go version, fix a single Go 1.20 issue in mount pkg; rm uneeded errorlint annotations

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

From the GitHub actions documentation [1];

> The macos-11 label has been deprecated and will no longer be available
> after 28 June 2024.

[1]: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Upgrades Node.js runtime from node16 to node20

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Only a single commit is fetched by default, for the ref/SHA that triggered
the workflow. Set fetch-depth: 0 to fetch all history for all branches and
tags.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Commit d6a5209 removed most errorlint
annotations, with one remaining.

golangci-lint v1.59.1 comes with errorlint v1.5.2, which contains
the fix [1] whitelisting all errno comparisons for errors coming from
x/sys/unix. Remove the annotation that is no longer needed.

[1]: polyfloyd/go-errorlint@c0e6cac

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Ubuntu 24.04 runners are now available (see [1])

[1]: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Add current macOS version, running on arm64  (see [1])

[1]: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Test with the current stable version of Go; keeping Go 1.17 as lower
boundary for now since it costs us little to keep supporting it.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

ci/gha: add Go 1.22, update various actions, add macos-12, ubuntu-24.04

- Fix references to EvalSymlinksInScope, which is non-exported
- Add relevant links to original implementation / introduction of the
  code to allow comparing the fork with the original upstream code.
- Move GoDoc from the non-exported evalSymlinksInScope function to
  the exported FollowSymlinkInScope.
- Remove the README.md, instead using doc.go to provide the package
  description.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

symlink: touch-up documentation