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

Adding LXXLCV versification #11

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

Conversation

viktor-zhuromskyy
Copy link

Adding LXXLCV versification where added DID for quick merge test

@@ -150,6 +150,8 @@ John.Short=Ин
John.Alt=ин,инн
Acts.Full=Деяния
Acts.Short=Деян
Did.Full=Дидахе
Copy link

Choose a reason for hiding this comment

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

translations are made via Transifex so this will be overwritten. See https://github.com/AndBible/and-bible/wiki/Translating-User-Interface

Copy link
Author

Choose a reason for hiding this comment

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

Does that mean the code merge not accepted?

Copy link

Choose a reason for hiding this comment

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

No, it's okay, just saying. But translations need to be done also in transifex.

Copy link
Author

Choose a reason for hiding this comment

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

So, I think I properly introduced LXXLCV versification in the jsword form, but the system does fail gradle tests.

Copy link

Choose a reason for hiding this comment

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

What you could do is to try to build AndBible with your version and try it out with your m odule.

Copy link

Choose a reason for hiding this comment

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

Did you manage to test this?

gradle throws thousand of TEST FAIL errors. I thought it is my clone repo errors, but when I ran ./gradlew test on your repo, same errors appeared.

It looks like CI has been successfull, but not in your branch. Though I have noticed that JSword CI sometimes gives false negatives. https://github.com/AndBible/jsword/actions/runs/4090858645/jobs/7054692044

Copy link
Author

Choose a reason for hiding this comment

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

Did you manage to test this?

gradle throws thousand of TEST FAIL errors. I thought it is my clone repo errors, but when I ran ./gradlew test on your repo, same errors appeared.

It looks like CI has been successfull, but not in your branch. Though I have noticed that JSword CI sometimes gives false negatives. https://github.com/AndBible/jsword/actions/runs/4090858645/jobs/7054692044

What command should I run to test my edits besides running ./gradlew test?

@tuomas2
Copy link

tuomas2 commented Feb 7, 2023

Did you manage to test this?

@viktor-zhuromskyy
Copy link
Author

Did you manage to test this?

gradle throws thousand of TEST FAIL errors.
I thought it is my clone repo errors, but when I ran ./gradlew test on your repo, same errors appeared.

@tuomas2
Copy link

tuomas2 commented Feb 8, 2023

Try ./gradlew test -Ptests=true

If you drop me email I can send you test modules package that is needed to run tests locally. It is automatically installed in CI.

@viktor-zhuromskyy
Copy link
Author

Try ./gradlew test -Ptests=true

If you drop me email I can send you test modules package that is needed to run tests locally. It is automatically installed in CI.

Updated and installed all dependencies on my Ubuntu machine, ran the ./gradlew test -Ptests=true, but still getting bunch of errors trying to run the command on your own repos in order to see if I am setup all good, but to no avail yet.

Still waiting for your test script.

I really want to contribute but I need to set up proper environment.

@tuomas2
Copy link

tuomas2 commented Feb 9, 2023

I emailed you test modules.

@viktor-zhuromskyy
Copy link
Author

How to cancel on my side the failed commit?

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

Successfully merging this pull request may close these issues.

2 participants