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

#3132171: Implement List #29

Merged
merged 1 commit into from
Feb 17, 2022
Merged

#3132171: Implement List #29

merged 1 commit into from
Feb 17, 2022

Conversation

veej
Copy link
Member

@veej veej commented Feb 16, 2022

Closes #3132171

Test Plan

tests performed

A Test Plan is used to show what you tested to make sure your code works fine. You should write here all that you did to test, and provide some results of your testing.

These results can be screenshots, query results, or even just pointers to unit tests that successfully passed.

tests not performed (domain coverage)

At times not everything can be tested, and writing what hasn't been tested is just as important as writing what has been tested.

An example of partial test is a field displaying 4 possible values. If 3 values are tested, with screenshots, and 1 is not, then it should be mentioned here.

@kanbanbot kanbanbot added the WIP label Feb 16, 2022
@veej veej changed the base branch from main to 3115561-colors_update February 16, 2022 15:06
@veej veej force-pushed the 3115561-colors_update branch from aa7bc66 to c98d76f Compare February 17, 2022 10:58
@veej veej force-pushed the 3132171-implement_list branch from 72c68e8 to c7edc5e Compare February 17, 2022 11:08
@veej veej requested a review from gabro February 17, 2022 11:08
@kanbanbot kanbanbot added in review and removed WIP labels Feb 17, 2022
@veej veej requested a review from Phyele February 17, 2022 11:55
Base automatically changed from 3115561-colors_update to main February 17, 2022 14:13
@veej veej merged commit 91410e3 into main Feb 17, 2022
@veej veej deleted the 3132171-implement_list branch February 17, 2022 14:59
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.

3 participants