Skip to content

feat(build status): get list of build status#52

Open
kishore03109 wants to merge 2 commits intofeat/getBuildTimesfrom
feat/getBuildStatus
Open

feat(build status): get list of build status#52
kishore03109 wants to merge 2 commits intofeat/getBuildTimesfrom
feat/getBuildStatus

Conversation

@kishore03109
Copy link
Copy Markdown
Contributor

Problem

We dont have an easy way to get the build status. This is a modified version of the build times script to just get the status.

NOTE: There could have been optimisations made to this to make both scripts use the same function. (there exists alot of similarity). Opting against this as

  1. this script is quite easy
  2. this is not be code, highly unlikely to be worked on in the future

@kishore03109 kishore03109 requested a review from a team September 7, 2023 01:31
Copy link
Copy Markdown
Contributor

@dcshzj dcshzj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just an idea for enhancement (no need in this PR): Should we do for both master and staging in a single run?

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