Skip to content

Refactor buildtest codebase#1763

Merged
shahzebsiddiqui merged 30 commits intodevelfrom
refactor_codebase
Apr 30, 2024
Merged

Refactor buildtest codebase#1763
shahzebsiddiqui merged 30 commits intodevelfrom
refactor_codebase

Conversation

@shahzebsiddiqui
Copy link
Member

No description provided.

pre-commit-ci bot and others added 15 commits April 26, 2024 01:04
… ('buildtest buildspec maintainers --breakdown')
…ldspec find invalid) by using methods

like create_table, print_table for printing.

Add --pager and --row-count option to the command menu since implementation was present but option was not.
fix bug in regression test where method name was not correct.
… maintainers' command.

add option --row-count for 'buildtest buildspec find invalid' to show count of invalid tests
add implementation for these options

Add bash completion support for new options
fix bug in regression test for viewing executors
remove unused variables in method handle_executors_command
…ethod name create_table used

for generating tables.
Remove option --theme from 'buildtest buildspec summary' since its not implemented in codebase
… helper method create_history_table used for printing the table history
The Report class will now pass values as **kwargs and there is a method set_report_parameters
that will assign instance variables passed on **kwargs.

Make use of nested for loops and combining if statements together to help reduce lines of codes where possible
…must match the actual arguments for buildtest report command
@shahzebsiddiqui shahzebsiddiqui merged commit a8a1162 into devel Apr 30, 2024
@shahzebsiddiqui shahzebsiddiqui deleted the refactor_codebase branch April 30, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant