Skip to content
This repository has been archived by the owner on Apr 22, 2021. It is now read-only.

Add CI via Github Actions #267

Open
wants to merge 109 commits into
base: master
Choose a base branch
from
Open

Conversation

MCOfficer
Copy link
Contributor

This PR adds an action run that generates a signed .jar and .exe on every commit.

In order to make the CI use your certificate, you need to:

  • Encrypt your keystore and replace keystore.gpg with it. See steps 1-4
  • Under Settings -> Secrets, add
    • CERT_ALIAS - your certificate's alias in the keystore
    • KEYSTORE_ENCRYPTION_PASS - the passphrase used to encrypt the keystore
    • KEYSTORE_PASS - the keystore's password

Tehrasha and others added 30 commits January 28, 2019 02:07
Corrected and clarified the 'MISSING RENDER' error case.
Make config file loader take ram in MB for consistency
… allow by scheduler depending on the past error, current settings (ram, cpu, gpu,...)
Fix grammar and consistency of README
luguina and others added 22 commits June 14, 2020 12:01
)

* Feature: show downloads in a single-line in text UIs
The text UI gets quite messy during the render process, as it takes at least one extra line of the screen per second. This PR fixes that issue and also shows an ASCII bar to show the render progress.
)

* Fix: Send reset signal to server on job upload error
@MCOfficer MCOfficer marked this pull request as ready for review June 27, 2020 09:30
.github/workflows/ci.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@tuxun tuxun left a comment

Choose a reason for hiding this comment

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

I like the idea and PR is very nicely made.

svn_trunk_url="https://github.com/laurent-clouet/sheepit-client/trunk"
pwd=`pwd`
tmp_dir=`mktemp -d`
jvm_name="jdk-11.0.6+10-jre"
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this jvm version match the one declared in ci.yml via its /usr/lib/jvm/ root dir?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not required, JVM major versions are bytecode-compatible.

@laurent-clouet laurent-clouet force-pushed the master branch 2 times, most recently from 90c3325 to 7f1949e Compare October 21, 2020 11:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants