Skip to content

Feature/automated tests #2

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

Jump to bottom
Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Feature/automated tests #2

wants to merge 22 commits into from

Conversation

benhillier
Copy link
Collaborator

Snapshot of automated testing.
Connection to Jena.
Progress on /institutions/level 1

benhillier and others added 14 commits May 13, 2019 11:05
@benhillier
Initial Symfony Install
ac6a093
@benhillier
Add docker configuration
c5b6136
@benhillier
Ad docker README reminder
08aa612
@benhillier
Add Annotations and Ping controller
07da6b4
@benhillier
Add code validators
57befe5
@benhillier
Add list response of Institutions
45d13b6
@benhillier
Add template Institution files
d84ee52
Implement architectural sketch of OpenSkos API
6c55f26 
Added basic Institution Domain object and its repository.
Implemented Sparql Repository.
Added Ontologies from old OpenSKOS.
And many more...
@benhillier
Merge pull request #1 from antohaby/feature/14209-institutions
781d63f 
Implement architectural sketch of OpenSkos API
@benhillier
Snapshot. Completed Connection to Jena
4afe5b1
@benhillier
Added Pagination classes and testing
a180274
@benhillier
Added format to 'Pagination', and refactored it to 'ApiRequest'
251e350
@benhillier
Formatting
7a605e0
@benhillier
Added Encoders/Decoders for EasyRDF. Implemented some Content Types
76c8fa2
cdekok
cdekok approved these changes May 29, 2019
View reviewed changes
Copy link

@cdekok cdekok left a comment

Choose a reason for hiding this comment

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

Looks good to me, we probably do need another check for the copyright as they are missing on the files now. The old version of openskos had a git hook for this.
It's also a good idea to add a travis configuration so we can run the tests automatically.

@finwo finwo deleted the feature/automated-tests branch January 24, 2020 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cdekok cdekok cdekok approved these changes

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
@benhillier @cdekok @antohaby