Skip to content

ENG-1577 Updated ttl file with fields per evidence#919

Open
maparent wants to merge 2 commits intomainfrom
eng-1577-updated-ttl-file-with-fields-per-evidence
Open

ENG-1577 Updated ttl file with fields per evidence#919
maparent wants to merge 2 commits intomainfrom
eng-1577-updated-ttl-file-with-fields-per-evidence

Conversation

@maparent
Copy link
Collaborator

@maparent maparent commented Mar 27, 2026

@linear
Copy link

linear bot commented Mar 27, 2026

@supabase
Copy link

supabase bot commented Mar 27, 2026

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@maparent maparent requested a review from joelchan March 27, 2026 01:50
devin-ai-integration[bot]

This comment was marked as resolved.

graphite-app[bot]

This comment was marked as resolved.

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 new potential issues.

View 4 additional findings in Devin Review.

Open in Devin Review

rdfs:range dg:Claim;
rdfs:domain dg:Evidence.
dg:observationStatement a dgb:RelationDef;
rdfs:label "observation statement"@en;
Copy link
Contributor

Choose a reason for hiding this comment

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

🟡 New rdfs:label values violate sentence case rule — not capitalized

The AGENTS.md mandates "Use sentence case for documentation copy by default." Every other rdfs:label in this file capitalizes the first word (e.g., "Opposes", "Curated to", "Question"), but the two newly added labels "observation statement" and "observation of" are fully lowercase. In sentence case the first letter must be capitalized: "Observation statement" and "Observation of".

Suggested change
rdfs:label "observation statement"@en;
rdfs:label "Observation statement"@en;
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

rdfs:range dg:Evidence;
rdfs:domain dg:Claim.
dg:observationOf a dgb:RelationDef;
rdfs:label "observation of"@en;
Copy link
Contributor

Choose a reason for hiding this comment

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

🟡 New rdfs:label value violates sentence case rule — not capitalized

Same issue as the other new label: "observation of" at line 36 should be "Observation of" per the sentence case convention required by AGENTS.md and followed by all other labels in this file.

Suggested change
rdfs:label "observation of"@en;
rdfs:label "Observation of"@en;
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant