Skip to content

Commit 53486a1

Browse files
authored
Merge branch 'develop' into kate/1024
2 parents fe2952d + dbf8814 commit 53486a1

31 files changed

+517
-361
lines changed

.markdownlint.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,6 @@
44
"MD007": false,
55
"MD013": false,
66
"MD024": { "siblings_only": true },
7+
"MDO29": { "style": "one_or_ordered" },
78
"MD036": false
89
}

README.md

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,16 @@ gist is free and open to the public under the [Creative Commons Attribution 4.0
1414

1515
For more information on gist and to download previous versions, see the [Semantic Arts website](https://www.semanticarts.com/gist).
1616

17-
## Documentation
17+
## Setting up a Local gist Repository
18+
19+
- Clone the [gist GitHub repository](https://github.com/semanticarts/gist.git).
20+
- Run the script `./tools/setup.cmd`.
21+
- To work on gist, refer to the following documents:
22+
- [Contributing to gist](docs/Contributing.md)
23+
- [gist Style Guide](docs/gistStyleGuide.md)
24+
- [Contributor Quick Reference](docs/ContributorQuickReference.md)
25+
26+
## gist Documentation
1827

1928
We provide a number of resources for learning more about gist.
2029

@@ -23,12 +32,12 @@ We provide a number of resources for learning more about gist.
2332
Extensive documentation of gist is available in the [gist-doc repository](https://github.com/semanticarts/gist-doc). This repository contains documentation of the [Semantic Arts gist minimalist upper ontology](https://github.com/semanticarts/gist/tree/master) in three formats: narrative, graphical, and Widoco-style.
2433

2534
* **gist Constellations (eBook)**
26-
* The organization of the eBook is based around the idea of 'constellations' of classes and predicates that deal with related concepts.
35+
* The organization of the eBook is based around the idea of 'constellations' of classes and predicates that deal with related concepts.
2736
* epub and mobi formats
2837
* **gist Constellations (visualizations)**
2938
* A collection of PNG images built using the [Turtle Editor Viewer](http://semantechs.co.uk/turtle-editor-viewer/). Each image is made up of a single 'constellation' of classes as described above and shows the classes and their relationships, the literal annotations, and any anonymous classes used in class definitions.
3039
* **[Widoco](https://github.com/dgarijo/Widoco) documentation**
31-
40+
3241
### Additional Documentation
3342

3443
* **Videos:** We maintain a library of videos containing gist tutorials and recordings of our monthly gist Council meetings. You can find links to these videos on the [Semantic Arts website](https://www.semanticarts.com/gist/videos/), or directly access the entire catalog on [our YouTube channel](https://www.youtube.com/playlist?list=PLk2kJrehubb4dc3e5Db5Lvv9WMaOhV3V7).

docs/Contributing.md

Lines changed: 64 additions & 53 deletions
Large diffs are not rendered by default.

docs/ContributorQuickReference.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# gist Contributor Quick Reference
2+
3+
This is a quick reference guide to submitting a pull request (PR) to the gist GitHub repository. Refer to the [contribution guidelines](Contributing.md) for full details.
4+
5+
## Set up a Local GitHub Repository
6+
7+
- Clone the [gist GitHub repository](https://github.com/semanticarts/gist.git).
8+
- Run the script `./tools/setup.cmd`.
9+
- Check to make sure that `./tools/pre-commit` has been copied to `./.git/hooks` and is executable. This ensures, among other things, that the serializer `./tools/serializer/rdf-toolkit.jar` runs before each commit. The serializer converts files into a standard Turtle format in order to remove noise in the diffs. No ontology files should be committed to the repository without running the serializer.
10+
11+
## Create a Working Branch
12+
13+
- Create a working branch from the base (target) branch. This is typically `develop`, but there may be a release branch.
14+
- Name the branch with your username + slash + issue number + brief description. Example: `jsmith/546_documentation_updates`.
15+
16+
## Implement
17+
18+
- Change the status of the issue you are working on to "In Progress."
19+
- Work on one issue at a time unless they are closely interconnected.
20+
- Consult the [_gist Style Guide_](gistStyleGuide.md) to ensure you are following gist conventions and best practices during implementation.
21+
- The serializer `./tools/serializer/rdf-toolkit.jar` *must* be run before every commit. If you have followed the [repository setup instructions](#set-up-a-local-github-repository), this will occur automatically.
22+
- Add a release note as described in [_Contributing_](Contributing.md#release-notes).
23+
24+
## Submit a Pull Request
25+
26+
- When implementation is complete, [create a pull request](https://github.com/semanticarts/gist/pulls) to the base branch.
27+
- Add the text "Closes #nnn" (where nnn is the issue number) to the description field. (If there are multiple issues, you must use the keyword "Closes" before each one.)
28+
- Assign the PR to yourself.
29+
- Add the PR to the same project as the issue. Set its status to "In Review."
30+
- Set the issue status to "In Review."
31+
- Submit the PR.
32+
- Resolve any conflicts with the base branch.
33+
- Assign one or more reviewers.
34+
- Address reviewer comments.
35+
- For straightforward changes, resolve the comment when fixed.
36+
- Re-request reviews.

docs/MajorVersionMigration.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ the release package will include queries to facilitate the migration of existing
88

99
For changes that are not amenable to automatic migration, queries will be provided that
1010
report the use of deleted or modified classes and properties so that mitigation measures
11-
can be initiated.
11+
can be initiated. For example, some releases may require manual intervention when a property is being replaced by its inverse (in class expressions, restrictions, and definitions of subproperties).
1212

1313
Every major version of `gist` (starting with 10.0.0) will add a sub-directory under the
1414
`migration` directory containing the queries and two migration scripts:

docs/ReviewerGuidelines.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Guidelines for PR Reviewers
2+
3+
- PR reviewers should ensure that the guidelines for submitting a PR have been followed; see [guidelines for submitting a PR in _Contributing_](./Contributing.md#submitting-a-pull-request-pr).
4+
- If any of the following problems exist, the reviewer may opt to fix them or return the PR to the author:
5+
- The PR has not been assigned to the same project as the issue being addressed.
6+
- The PR and the issue have not been assigned status "In Review."
7+
- The description does not list the issues closed by the PR, using the phrase "Closes #nnn" where nnn is the issue number. If more than one issue is closed, each should appear on a separate line preceded by the word "Closes."
8+
- The PR is not submitted to the appropriate branch (usually `develop`, but in some cases there may be a release branch).
9+
- The PR should be returned to the author for any of the following problems:
10+
- There is no release note or explanation in the PR description of why none has been included.
11+
- The release note does not adequately reflect the contents of the PR or does not conform to the [guidelines](./Contributing.md#release-notes)
12+
- The serializer has not been run or the diffs are noisy for some other reason.
13+
- The changes do not conform to the agreed upon implementation described at the bottom of the issue.
14+
- The ontology is not logically consistent.
15+
- There are conflicts with the base (target) branch.
16+
- Some checks have not passed.
17+
- The changes included in the PR are not all within the scope of the issue.
18+
- Commenting on the PR:
19+
- Review the [GitHub documentation on commenting on PRs](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request) for general information.
20+
- Do _not_ commit your suggestions to the PR itself. Enter comments in the comment box for the author to review.
21+
- It is helpful to enter minor suggestions such as small wording changes as GitHub suggestions, allowing the author to accept and commit the suggestion with a button click. You can do this by clicking on the small +- icon to the right of the word "Preview" in the comment box. See [item 7 in the GitHub documentation](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-comments-to-a-pull-request).

0 commit comments

Comments
 (0)