-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Open
Labels
area/cikind/feature-requestDescribes new functionalityDescribes new functionalitypriority/after-1.5Leave for after the Cirq 1.5 releaseLeave for after the Cirq 1.5 releasetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on
Description
Is your feature request related to a use case or problem? Please describe.
Add a CI workflow to lint & check YAML files.
Describe the solution you'd like
Per discussions on issue #6896, this is a tracking issue to remind us to do 2 things:
- Add yamllint to the CI workflow and the
check/all
script - Try if possible to auto-fix lint (perhaps using the Google tool
yaml_format
)
[optional] Describe alternatives/workarounds you've considered
Currently, we have a configuration file for yamllint. It needs to be invoked manually, which is not great, but better than nothing.
What is the urgency from your perspective for this issue? Is it blocking important work?
P2 - we should do it in the next couple of quarters
Metadata
Metadata
Assignees
Labels
area/cikind/feature-requestDescribes new functionalityDescribes new functionalitypriority/after-1.5Leave for after the Cirq 1.5 releaseLeave for after the Cirq 1.5 releasetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on