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

Fix short circuting of && and || in macros #3655

Merged
merged 1 commit into from
Jan 16, 2024

Fix short circuting of && and || in macros

c87959d
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

Fix short circuting of && and || in macros #3655

Fix short circuting of && and || in macros
c87959d
Select commit
Loading
Failed to load commit list.
GitHub Actions / vale completed Jan 16, 2024 in 0s

reviewdog [vale] report

reported by reviewdog 🐶

Findings (3)

docs/src/packages/Loraine.md|78 col 20| [Google.Ellipses] In general, don't use an ellipsis.
docs/src/packages/Ipopt.md|256 col 33| [Google.Colons] ': L' should be in lowercase.
docs/src/packages/Manopt.md|36 col 35| [Google.Colons] ': O' should be in lowercase.

Filtered Findings (0)

Annotations

Check warning on line 78 in docs/src/packages/Loraine.md

See this annotation in the file changed.

@github-actions github-actions / vale

[vale] docs/src/packages/Loraine.md#L78

[Google.Ellipses] In general, don't use an ellipsis.
Raw output
{"message": "[Google.Ellipses] In general, don't use an ellipsis.", "location": {"path": "docs/src/packages/Loraine.md", "range": {"start": {"line": 78, "column": 20}}}, "severity": "WARNING"}

Check warning on line 256 in docs/src/packages/Ipopt.md

See this annotation in the file changed.

@github-actions github-actions / vale

[vale] docs/src/packages/Ipopt.md#L256

[Google.Colons] ': L' should be in lowercase.
Raw output
{"message": "[Google.Colons] ': L' should be in lowercase.", "location": {"path": "docs/src/packages/Ipopt.md", "range": {"start": {"line": 256, "column": 33}}}, "severity": "WARNING"}

Check warning on line 36 in docs/src/packages/Manopt.md

See this annotation in the file changed.

@github-actions github-actions / vale

[vale] docs/src/packages/Manopt.md#L36

[Google.Colons] ': O' should be in lowercase.
Raw output
{"message": "[Google.Colons] ': O' should be in lowercase.", "location": {"path": "docs/src/packages/Manopt.md", "range": {"start": {"line": 36, "column": 35}}}, "severity": "WARNING"}