Skip to content

Sync linting completed#3048

Open
Scott-Owen-James wants to merge 11 commits into
metomi:masterfrom
Scott-Owen-James:3034
Open

Sync linting completed#3048
Scott-Owen-James wants to merge 11 commits into
metomi:masterfrom
Scott-Owen-James:3034

Conversation

@Scott-Owen-James

@Scott-Owen-James Scott-Owen-James commented Jun 1, 2026

Copy link
Copy Markdown

closes #3034

@oliver-sanders

Copy link
Copy Markdown
Member

Hmmm:

  • check-contributor/test: failing, but not because of this PR.
  • test/lint: also failing for what appear to be unrelated issues
  • test/test: appears to be failing for genuine reasons
+  File "/home/runner/micromamba/envs/rose-test/lib/python3.12/site-packages/metomi/rose/macros/compulsory.py", line 240, in _generate_aliases_for_sections
+    basic_sections = set(
+                     ^^^^
+TypeError: set expected at most 1 argument, got 2

Comment thread metomi/rose/metadata_gen.py Outdated
Comment thread metomi/rose/config_editor/pagewidget/table.py Outdated
@oliver-sanders

Copy link
Copy Markdown
Member

(re-running the failing checks...)

Comment thread pyproject.toml Outdated
Comment thread pyproject.toml
Comment thread pyproject.toml
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

Successfully merging this pull request may close these issues.

sync linting from cylc-flow

2 participants