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: Tidy up blank lines in 'explain' output #2537

Merged
merged 20 commits into from
Dec 26, 2023
Merged

Conversation

claremacrae
Copy link
Collaborator

Description

Tidy up the blank lines in explain output:

  • Remove some duplicate blank lines
  • Add some missing blank lines

See differences in the test output.

Motivation and Context

Make the code easier to maintain, before adding some more explanations.

How has this been tested?

  • Automated tests
  • Exploratory testing

Screenshots (if appropriate)

Types of changes

Changes visible to users:

  • Bug fix (prefix: fix - non-breaking change which fixes an issue)
  • Documentation (prefix: docs - improvements to any documentation content for users)

Internal changes:

  • Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
  • Tests (prefix: test - additions and improvements to unit tests and the smoke tests)

Checklist

Terms

@claremacrae claremacrae added the scope: explain About the `explain` instruction which generates explanations of tasks query blocks label Dec 26, 2023
@claremacrae claremacrae merged commit 0d51c2d into main Dec 26, 2023
2 checks passed
@claremacrae claremacrae deleted the tidy-explain-output branch December 26, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: explain About the `explain` instruction which generates explanations of tasks query blocks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant