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

New warning from conda-forge linting-process / souschef #61

Open
willend opened this issue Nov 28, 2024 · 0 comments
Open

New warning from conda-forge linting-process / souschef #61

willend opened this issue Nov 28, 2024 · 0 comments

Comments

@willend
Copy link

willend commented Nov 28, 2024

Hi there,

I have a couple of conda-forge feedstocks (mcstas-suite,mcxtrace-suite) that build multiple packages through a single feedstock-recipe, and were we use "non-standard" naming of the buildscripts for Unix and Windows, namely e.g.

outputs:

  - name: mcstas-core
    script: install-files-core.sh  # [unix]
    script: install-files-core.bat  # [win]

I noticed that conda-souschef (in fact maybe a new YaML release?) recently has started complaining / warning about this:

2024-11-26 08:24:37,400 WARNING conda_smithy.linter.lints || Error parsing recipe with conda-souschef: DuplicateKeyError('while constructing a mapping', in "<unicode string>", line 19, column 5:

[](https://github.com/conda-forge/conda-forge-webservices/actions/runs/12026574210/job/33525822640#step:5:81)- name: mcxtrace-core

[](https://github.com/conda-forge/conda-forge-webservices/actions/runs/12026574210/job/33525822640#step:5:82)^ (line: 19), 'found duplicate key "script" with value "install-files-core.bat" (original value: "install-files-core.sh")', in "<unicode string>", line 21, column 5:

[](https://github.com/conda-forge/conda-forge-webservices/actions/runs/12026574210/job/33525822640#step:5:83)script: install-files-core.bat ...

[](https://github.com/conda-forge/conda-forge-webservices/actions/runs/12026574210/job/33525822640#step:5:84)^ (line: 21), '\n To suppress this check see:\n http://yaml.readthedocs.io/en/latest/api.html#duplicate-keys\n ', ' Duplicate keys will become an error in future releases, and are errors\n by default when using the new API.\n ')

Screenshot 2024-11-26 at 09 28 06

After a discussion on Zulip I was asked to file an issue here.

@willend willend changed the title New warning from conda-forge listing / souschef New warning from conda-forge linting-process / souschef Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant