petkostas commented on PR #8598: URL: https://github.com/apache/incubator-devlake/pull/8598#issuecomment-3386193653
> @petkostas Thank for the information. Testing new features for all data sources is time-consuming, sometimes impossible due to paywalls or data availability. I think it is OK to push it out when it is working properly on 1 data source. > > Adding new statuses for all data sources should be OK as long as it doesn't break any existing dashboards IMO. @Startrekzky Do you recall any dashboards depending on the `cicd_pipelines.status` being `SUCCESS`, `FAILURE`, and `` only? In other words, would any dashboard break if new statuses `CANCELED` and `SKIPPED` were introduced? Thanks @klesh I think it's best to move with GitHub only and then gradually introduce the others through maybe contributors who have access to the other data sources. I will re-factor the existing PR to include only GitHub related changes, I think it will also help the changes reach the main branch faster 😃 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
