Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions zuul.d/project.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@
check:
jobs:
- openstack-tox-pep8

check-review:
jobs:
- tenks-ubuntu-noble
- tenks-rocky-9
# Disable zuul checks until they are fixed.
# check-review:
# jobs:
# - tenks-ubuntu-noble
# - tenks-rocky-9
Comment on lines +7 to +11
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

Disabling checks temporarily is a reasonable approach. To improve long-term maintainability, it's a best practice to include a reference to a tracking issue (e.g., a GitHub issue) in the comment. This provides context for why the checks are disabled and helps track when they should be re-enabled.

Additionally, the current commenting style for the YAML block could be difficult to uncomment correctly later. A cleaner approach is to prepend # to each line, preserving the original indentation, which makes uncommenting straightforward.

# Disable zuul checks until they are fixed (see #<issue-number>).
#    check-review:
#      jobs:
#        - tenks-ubuntu-noble
#        - tenks-rocky-9


gate:
jobs:
Expand Down
Loading