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
GitHub action to run builds #706
Conversation
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mickaelistria for starting this off. If you or others are interested in getting this complete here are my thoughts:
Perhaps you can show these working on your fork so that we can test the changes before we merge the PR?
There are also some additional steps that need to be done to check the PR, you can see current PR configuration in https://ci.eclipse.org/lsp4j/job/lsp4j-github-pullrequests/configure
Specifically needs:
- gradle
publish
and flags-PignoreTestFailures=true --refresh-dependencies --continue
mvn -f releng/pom.xml --batch-mode --update-snapshots clean install
- collecting and reporting test results
.github/workflows/validate.yml
Outdated
- name: Set up JDK 17 ☕ | ||
uses: actions/setup-java@v3 | ||
with: | ||
java-version: 17 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Java version should match what project uses, or at least a Java 8 (or 11, depending on state of #707) needs to be available too IIUC.
.github/workflows/validate.yml
Outdated
- main | ||
push: | ||
branches: | ||
- main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why limit the branches? I have seen projects do both. We initially did for CDT, but every release I have to keep adding new branches to this list.
b68882d
to
05f7f8e
Compare
For the moment, just a build that answers yes/no is most of what I can offer. I'm not familiar with Gradle and so on to do more. Also if I'm not mistaken, isn't the mvn only intended to do p2 stuff? |
yes mvn just builds the p2 repo. gradle puts test results to https://github.com/eclipse/lsp4j/blob/4bad511eb9471bef9aafc9ad8e7d9c55b6747a25/releng/build.Jenkinsfile#L84 |
Lets go with this for now, people can add to it as needed later. |
No description provided.