Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add loeffel-io/ls-lint #12669

Merged
merged 2 commits into from
May 31, 2023

Conversation

ponkio-o
Copy link
Sponsor Contributor

loeffel-io/ls-lint: An extremely fast directory and filename linter - Bring some structure to your project filesystem

$ aqua g -i loeffel-io/ls-lint

How to confirm if this package works well

Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.

Command and output

$ ls-lint --help
Usage of ls-lint-darwin-arm64:
  -config string
        ls-lint config file path (default ".ls-lint.yml")
  -debug
        write debug informations to stdout
  -version
        prints version information for ls-lint
  -warn
        write lint errors to stdout instead of stderr (exit 0)
  -workdir string
        change working directory before executing the given subcommand (default ".")

see. https://ls-lint.org/

[loeffel-io/ls-lint](https://github.com/loeffel-io/ls-lint): An extremely fast directory and filename linter - Bring some structure to your project filesystem
description: An extremely fast directory and filename linter - Bring some structure to your project filesystem
asset: ls-lint-{{.OS}}-{{.Arch}}
# error: opening tar archive for reading: wrapping file reader: gzip: invalid header
format: raw
Copy link
Sponsor Contributor Author

Choose a reason for hiding this comment

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

If use format: tar.gz an error occured by aqua.

$ aqua i --test
ERRO[0001] install the package                           aqua_version=2.8.0 env=darwin/arm64 error="opening tar archive for reading: wrapping file reader: gzip: invalid header" package_name=loeffel-io/ls-lint package_version=v2.0.0 program=aqua registry=standard

Copy link
Member

Choose a reason for hiding this comment

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

I'm taking a look.

https://github.com/loeffel-io/ls-lint/releases/tag/v2.0.0

The file has no problem.

$ tar vtzf ls-lint-darwin-arm64.tar.gz 
-r-xr-xr-x  0 0      0        1075  1  1  2000 LICENSE
-r-xr-xr-x  0 0      0     3155906  1  1  2000 ls-lint-darwin-arm64

$ tar xvzf ls-lint-darwin-arm64.tar.gz 
x LICENSE
x ls-lint-darwin-arm64

I could reproduce the issue.

$ aqua i
INFO[0000] create a symbolic link                        aqua_version=2.8.0 command=ls-lint env=darwin/arm64 program=aqua
INFO[0000] create a symbolic link                        aqua_version=2.8.0 command="ls-lint-{{.OS}}-{{.arch}}" env=darwin/arm64 program=aqua
INFO[0000] download and unarchive the package            aqua_version=2.8.0 env=darwin/arm64 package_name=loeffel-io/ls-lint package_version=v1.1.0 program=aqua registry=standard
INFO[0000] download and unarchive the package            aqua_version=2.8.0 env=darwin/arm64 package_name=loeffel-io/ls-lint package_version=v2.0.0 program=aqua registry=standard
INFO[0000] download and unarchive the package            aqua_version=2.8.0 env=darwin/arm64 package_name=loeffel-io/ls-lint package_version=v1.10.0-beta.1 program=aqua registry=standard
INFO[0000] download and unarchive the package            aqua_version=2.8.0 env=darwin/arm64 package_name=loeffel-io/ls-lint package_version=v1.0.0 program=aqua registry=standard
INFO[0001] downloading a checksum file                   aqua_version=2.8.0 env=darwin/arm64 package_name=loeffel-io/ls-lint package_version=v2.0.0 program=aqua registry=standard
ERRO[0001] install the package                           aqua_version=2.8.0 env=darwin/arm64 error="opening tar archive for reading: wrapping file reader: gzip: invalid header" package_name=loeffel-io/ls-lint package_version=v2.0.0 program=aqua registry=standard
FATA[0001] aqua failed                                   aqua_version=2.8.0 env=darwin/arm64 error="it failed to install some packages" program=aqua
packages:
  - type: github_release
    repo_owner: loeffel-io
    repo_name: ls-lint
    description: An extremely fast directory and filename linter - Bring some structure to your project filesystem
    asset: ls-lint-{{.OS}}-{{.Arch}}.{{.Format}}
    # error: opening tar archive for reading: wrapping file reader: gzip: invalid header
    format: tar.gz
    supported_envs:
      - darwin
      - linux
      - amd64
    checksum:
      type: github_release
      asset: checksums.txt
      algorithm: sha256
    version_constraint: semver(">= 2.0.0")
    version_overrides:
      - version_constraint: semver(">= 1.10.0-beta.1")
        asset: ls-lint-{{.OS}}
        format: raw
        overrides:
          - goos: linux
            goarch: arm64
            asset: ls-lint-{{.OS}}-{{.Arch}}
          - goos: darwin
            goarch: arm64
            asset: ls-lint-{{.OS}}-{{.Arch}}
        checksum:
          enabled: false
        files:
          - name: ls-lint-{{.OS}}-{{.arch}}
      - version_constraint: semver(">= 1.1.0")
        asset: ls-lint-{{.OS}}
        format: raw
        supported_envs:
          - darwin
          - amd64
        rosetta2: true
        checksum:
          enabled: false
        files:
          - name: ls-lint-{{.OS}}-{{.arch}}
      - version_constraint: semver("< 1.1.0")
        asset: ls-lint-{{.OS}}
        format: raw
        supported_envs:
          - linux/amd64
          - darwin
        rosetta2: true
        checksum:
          enabled: false
        files:
          - name: ls-lint-{{.OS}}-{{.arch}}

🤔

Copy link
Member

Choose a reason for hiding this comment

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

BTW, aqua i's --test option was deprecated at aqua v2.0.0.

$ aqua i --help | grep test
   --test                  This flag was deprecated and had no meaning from aqua v2.0.0. This flag will be removed in aqua v3.0.0. https://github.com/aquaproj/aqua/issues/1691 (default: false)

Copy link
Sponsor Contributor Author

Choose a reason for hiding this comment

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

BTW, aqua i's --test option was deprecated at aqua v2.0.0.

oh! thank you for teach me!

@suzuki-shunsuke suzuki-shunsuke added the enhancement New feature or request label May 31, 2023
@suzuki-shunsuke suzuki-shunsuke added this to the v4.12.0 milestone May 31, 2023
@suzuki-shunsuke
Copy link
Member

Thank you always!

@suzuki-shunsuke suzuki-shunsuke merged commit e7084e0 into aquaproj:main May 31, 2023
16 checks passed
@suzuki-shunsuke
Copy link
Member

@ponkio-o ponkio-o deleted the feat/loeffel-io/ls-lint branch May 31, 2023 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants