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

Remove unused Results field #1243

Closed
bobcatfish opened this issue Aug 26, 2019 · 0 comments · Fixed by #1425
Closed

Remove unused Results field #1243

bobcatfish opened this issue Aug 26, 2019 · 0 comments · Fixed by #1425
Assignees

Comments

@bobcatfish
Copy link
Collaborator

Expected Behavior

Our API should only contain fields we actually use.

Actual Behavior

The results field was part of our very first design and was imagine to contain info like links to logs and test results, but we have since not done anything with it. When we choose to revisit these topics we can add in fields as needed safely, but we should remove this field for now b/c it does nothing at all and the longer we leave it in, the greater the chances that someone will come to rely on it for some reason.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants