Skip to content

Replace rdf:type by a#35

Merged
samwaseda merged 5 commits into
refactorfrom
rdf_toA
Apr 2, 2025
Merged

Replace rdf:type by a#35
samwaseda merged 5 commits into
refactorfrom
rdf_toA

Conversation

@samwaseda

@samwaseda samwaseda commented Mar 31, 2025

Copy link
Copy Markdown
Collaborator

They mean the same thing, and it allows us to remove one line

@samwaseda
samwaseda changed the base branch from main to refactor March 31, 2025 07:54
@samwaseda
samwaseda requested review from Copilot and srmnitc March 31, 2025 07:56
Copilot AI reviewed Mar 31, 2025
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR aims to simplify the SPARQL query construction by replacing "rdf:type" with its shorthand "a" in the query string.

  • Removed explicit use of "rdf" namespace in the query.
  • Replaced "rdf:type" with "a" for brevity.

@samwaseda samwaseda mentioned this pull request Mar 31, 2025
Open
@samwaseda
samwaseda requested a review from aazocar April 1, 2025 10:12
aazocar
aazocar approved these changes Apr 2, 2025
View reviewed changes

@aazocar aazocar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍🏽

@samwaseda
samwaseda merged commit 58a91ab into refactor Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@aazocar aazocar aazocar approved these changes
@srmnitc srmnitc Awaiting requested review from srmnitc

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samwaseda @aazocar