Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

ANY23-611 -- general upgrades #321

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tballison
Copy link

Thanks for your contribution to Apache Any23! Your help is appreciated!

Before opening the pull request, please verify that

  • there is an open issue on the Any23 issue tracker which describes the problem or the improvement. We cannot accept pull requests without an issue because the change wouldn't be listed in the release notes.
  • the issue ID (ANY23-XXXX)
    • is referenced in the title of the pull request
    • and placed in front of your commit messages surrounded by square brackets ([ANY23-XXXX] Issue or pull request title)
  • commits are squashed into a single one (or few commits for larger changes)
  • Java source code follows formatter-maven-plugin formatting rules
  • Any23 is successfully built and unit tests pass by running mvn clean test
  • there should be no conflicts when merging the pull request branch into the recent master branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled master branch.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Any23 in general, please sign up for the Any23 mailing list. Thanks!

@tballison
Copy link
Author

tballison commented May 24, 2023

This will have to be updated after pr #320 . I'm happy to do so if this general PR seems useful.

@@ -181,6 +182,22 @@ public Literal createLiteral(String pref, IRI value) {
return wrappedFactory.createLiteral(pref, value);
}

Copy link
Author

Choose a reason for hiding this comment

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

I don't know what I'm doing here. This needs to be reviewed and fixed by someone who does.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant