Skip to content

fix bug with boolean values in pyproject.toml#102

Merged
modehnal merged 4 commits intobehave-contrib:mainfrom
Kory291:main
Dec 16, 2025
Merged

fix bug with boolean values in pyproject.toml#102
modehnal merged 4 commits intobehave-contrib:mainfrom
Kory291:main

Conversation

@Kory291
Copy link
Copy Markdown
Contributor

@Kory291 Kory291 commented Dec 11, 2025

I ran tox but ruff seems to fail there with ruff .:

ruff: commands[0]> ruff . --show-source
error: unrecognized subcommand '.'

Otherwise I think I got everything.

with thead(
onclick="toggle_hash("
f"'table_{PrettyHTMLFormatter.table_number}')",
onclick=f"toggle_hash('table_{PrettyHTMLFormatter.table_number}')",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think that got changed because I ran ruff format ...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah please do not change anything else, revert the other changes made by ruff, if it still complains in here I will resolve it on my side.

@modehnal modehnal self-assigned this Dec 11, 2025
with thead(
onclick="toggle_hash("
f"'table_{PrettyHTMLFormatter.table_number}')",
onclick=f"toggle_hash('table_{PrettyHTMLFormatter.table_number}')",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah please do not change anything else, revert the other changes made by ruff, if it still complains in here I will resolve it on my side.

@Kory291
Copy link
Copy Markdown
Contributor Author

Kory291 commented Dec 16, 2025

I reverted the last commit and implemented the changes again.

@modehnal
Copy link
Copy Markdown
Collaborator

Looking good, merging.

@modehnal modehnal merged commit b8590fe into behave-contrib:main Dec 16, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants