Skip to content

update to test #3388

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
joejstuart wants to merge 1 commit into
base: main
Choose a base branch
from
Open

update to test #3388

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 2 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# `ec` a command line client for verifying artifacts and evaluating policies

## testing changes

@fullsend-ai-review fullsend-ai-review Bot Jul 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[high] intent-alignment

No linked issue or authorization for this change. The PR has no body, no linked issue, and the title "update to test" combined with the branch name "test-ta" indicate this is a scratch/test PR rather than authorized work. The content-free heading serves no documented purpose and there is no traceability to any planned work item.

Suggested fix: If this PR serves a legitimate purpose (e.g., testing CI pipelines or webhook integrations), document that in the PR body and link to the relevant issue. Otherwise, close the PR without merging.

@fullsend-ai-review fullsend-ai-review Bot Jul 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[medium] scope-coherence

Content-free placeholder heading degrades README structure. Inserting a meaningless "## testing changes" heading between the project title and the project description breaks the document logical structure and narrative flow. The README is the project public-facing documentation.

Suggested fix: Remove the heading. If changes to the README are needed, add substantive content that follows the existing document structure.

@fullsend-ai-review fullsend-ai-review Bot Jul 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[low] naming-conventions

Heading uses lowercase ("## testing changes") which is inconsistent with every other level-2 heading in the file (## Building, ## Testing, ## Linting, ## Demo, ## Troubleshooting). The established convention is title case.

Suggested fix: Use title case: "## Testing Changes".


The `ec` tool is used to evaluate Conforma policies for Software
Supply Chain. Various sub-commands can be used to assert facts about an artifact
such as:
Expand Down
Loading