bazelbuild / bazelbuild/continuous-integration

buildifier checks should use the json output rather than parsing stdout/stderr

Open
#966 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
302
Forks
194
Avg merge
1d 19h
Merged PRs (30d)
41

Description

As mentioned by @vladmos in #943, the integration should likely be updated to use `--mode=check --format=json` rather than parsing stdout/stderr of a buildifier run, that should ensure bugs because out output formatting changes don't happen again.

Contributor guide

Open the contributing guide

Research direction

The issue points to the buildifier integration and the #943 discussion; start by locating where a buildifier run's stdout/stderr is parsed. Check how the --mode=check --format=json output should be consumed, then verify the CI check still reports formatting failures correctly without depending on output formatting.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
ci-cd
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.