Skip to content

Bump splunk-appinspect from 4.3.0 to 4.3.1 - #148

Merged
oschwald merged 1 commit into
mainfrom
wstorey/bump-appinspect-4.3.1
Aug 25, 2026
Merged

Bump splunk-appinspect from 4.3.0 to 4.3.1#148
oschwald merged 1 commit into
mainfrom
wstorey/bump-appinspect-4.3.1

Conversation

@horgh

@horgh horgh commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

AppInspect 4.3.0 added check_for_custom_mako_templates as a FUTURE failure. It flags UCC's appserver/templates/base.html by path only (the file is plain HTML), and its exit code 104 turns the appinspect lint red.

4.3.1 (released 2026-08-19) demotes this check, plus the two other 4.3.0 FUTURE checks, from FailMessage to WarningMessage. Warnings exit 103, which .precious.toml already lists in ok-exit-codes, so precious lint --command appinspect passes again with no config change.

The underlying UCC base.html issue is still unfixed upstream, so this may return when AppInspect 4.4.0 enforces it.

Only uv.lock changes; the >=4.3.0 pin stays.

🤖 Generated with Claude Code

AppInspect 4.3.0 added check_for_custom_mako_templates as a FUTURE
failure. It flags UCC's appserver/templates/base.html by path only, and
its exit code 104 fails the lint. 4.3.1 demotes this check (and two
other 4.3.0 FUTURE checks) to warnings, which precious already accepts.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 24, 2026 19:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d116359e-5b96-412e-9577-7268bae04c80


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@oschwald
oschwald merged commit a74841b into main Aug 25, 2026
10 checks passed
@oschwald
oschwald deleted the wstorey/bump-appinspect-4.3.1 branch August 25, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants