Set TERM to dumb in CI

[mernst]

Summary by CodeRabbit

• Chores
  • Updated CI environment settings to standardize terminal handling across jobs, improving build log clarity and stability.
  • Modernized CI workflows by removing a deprecated version declaration, aligning with current platform conventions and reducing maintenance overhead.
  • No changes to product features, performance, or UI; no action required from users.
  • Slight improvements to build reliability.

[coderabbitai]

📝 Walkthrough

Walkthrough

Added TERM=dumb to CircleCI job environments via config and m4 macros, and removed the explicit workflows version line. No job steps or control flow were changed.

Changes

Cohort / File(s)	Summary of changes
CircleCI pipeline config /.circleci/config.yml	Added environment variable TERM: dumb to jobs; removed explicit workflows version: 2 line; no step/command changes.
CircleCI m4 templates /.circleci/config.yml.m4	Added TERM: dumb to canary-job environments; removed workflows version: 2 line.
CircleCI boilerplate defs /.circleci/defs.m4	Added TERM=dumb to circleci_boilerplate environment alongside CIRCLE_COMPARE_URL; no other modifications.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Reduce CircleCI boilerplate #663 — Updates the same CircleCI boilerplate by adding TERM: dumb and removing the workflows version line, touching the same macros and config paths.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "Set TERM to dumb in CI" is a concise, accurate summary of the primary change in the diff, which adds TERM: dumb to CircleCI job environments in .circleci/config.yml, .circleci/config.yml.m4, and .circleci/defs.m4; it clearly communicates the main intent to reviewers.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4980d95 and 613add8.

📒 Files selected for processing (3)

• .circleci/config.yml (69 hunks)
• .circleci/config.yml.m4 (1 hunks)
• .circleci/defs.m4 (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)

• GitHub Check: codespecs.daikon (typecheck_bundled_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (nonquick_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (kvasir_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (typecheck_latest_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (misc_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (nontxt_ubuntu_jdk25)
• GitHub Check: codespecs.daikon (quick_ubuntu_jdk25)
• GitHub Check: build

🔇 Additional comments (3)

.circleci/config.yml.m4 (1)

14-14: Consistent with macros; canary-jobs needed explicit TERM.

Looks correct and aligned with circleci_boilerplate so all jobs get TERM=dumb.

.circleci/config.yml (1)

13-13: Bulk TERM=dumb additions: LGTM.

Generated config reflects the template change and adds TERM consistently across jobs.

If this file is generated, ensure it’s rebuilt from the m4 sources before merge to prevent drift. The verification script in my other comment checks that every job exposes TERM=dumb.

Also applies to: 23-23, 43-43, 63-63, 83-83, 103-103, 123-123, 144-144, 164-164, 184-184, 204-204, 224-224, 244-244, 265-265, 285-285, 303-303, 322-322, 341-341, 360-360, 380-380, 401-401, 422-422, 443-443, 464-464, 485-485, 507-507, 529-529, 551-551, 573-573, 595-595, 617-617, 642-642, 664-664, 689-689, 711-711, 736-736, 758-758, 783-783, 805-805, 831-831, 851-851, 871-871, 891-891, 911-911, 931-931, 952-952, 972-972, 992-992, 1012-1012, 1032-1032, 1052-1052, 1073-1073, 1092-1092, 1111-1111, 1130-1130, 1149-1149, 1168-1168, 1188-1188, 1209-1209, 1230-1230, 1251-1251, 1272-1272, 1293-1293, 1315-1315, 1337-1337, 1359-1359, 1381-1381, 1403-1403, 1425-1425

.circleci/defs.m4 (1)

13-13: Approve — TERM=dumb added; no terminal-dependent calls found

Confirmed .circleci/config.yml contains TERM: dumb entries (set via .circleci/defs.m4's circleci_boilerplate). Repo scan found no runtime uses of tput, stty, infocmp, tty, setaf, sgr0, or other termcap/ANSI-control commands — matches were documentation or non-executable/Makefile lines (e.g. tests/junit5/Makefile disables ANSI). No changes required.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}