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

cypress-run to depend on a maven build #2970

Merged
merged 1 commit into from
Jan 23, 2025
Merged

cypress-run to depend on a maven build #2970

merged 1 commit into from
Jan 23, 2025

Conversation

afs
Copy link
Member

@afs afs commented Jan 22, 2025

This PR does two things:

  • cyrpress-run is made to run after the build job. It appears that the jobs run in parallel at the moment - I see the Cypress runs finish before the build step has. The Cypress job is quick.

  • Change the build step to do a "dev" build. Currently, it does not build from the start of the Jena build. If there has been a change in earlier dependency, then the Fuseki steps fail.

If this become smooth, we should consider combining into a PR workflow.


By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.

@afs
Copy link
Member Author

afs commented Jan 22, 2025

Bumping the timeout on the maven step ...

Copy link
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

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

Good idea to run cypress after unit/build, that should avoid some unnecessary e2e tests. +1 🎉

@afs afs merged commit 4c6ac6c into apache:main Jan 23, 2025
@afs afs deleted the ui-workflow branch January 23, 2025 11:01
@afs
Copy link
Member Author

afs commented Jan 23, 2025

The fun thing is that it is the updated workflow in the PR is the one run for the PR.

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