Skip to content
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.

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

Update docker-deployment.yml #230

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Update docker-deployment.yml #230

merged 1 commit into from
Dec 12, 2023

Conversation

dschiese
Copy link
Collaborator

@dschiese dschiese commented Dec 7, 2023

@anbo-de
The deployment action still used the stardog env variable. Changed it with respect to your changes in 0a810e2

Hopefully everything is configured now

Changed env variable from Stardog to Virtuoso
@dschiese
Copy link
Collaborator Author

dschiese commented Dec 7, 2023

The Maven action failed somehow? Any suggestions?

@dschiese
Copy link
Collaborator Author

@anbo-de Can be merged

@anbo-de anbo-de self-assigned this Dec 12, 2023
@anbo-de anbo-de merged commit fb1e2ef into WDAqua:master Dec 12, 2023
4 checks passed
@dschiese
Copy link
Collaborator Author

It stills fails somehow. The build ends successfully and within the build-process no errors occur?

@anbo-de
Copy link
Member

anbo-de commented Dec 12, 2023

It stills fails somehow. The build ends successfully and within the build-process no errors occur?

The reason was a process that does not return the code 0.

The set -e command that every command completed successfully, but, the error message is not visible. I removed it: ade52ee

Works now: https://github.com/WDAqua/Qanary/actions/runs/7175970943 😄

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

Successfully merging this pull request may close these issues.

2 participants