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

Copy resources directly without zip #653

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

vChavezB
Copy link
Contributor

Instead of using zip files, use the source files in the resources directory. This allows to track differences easily over git as zip files cannot be compared between commits easily.

In specific I did this as I am testing the swrl rules xslt files (see #652 ) and found out that the zip file is used instead of the src/main/resources/lode folder.

@vChavezB vChavezB mentioned this pull request Dec 10, 2023
@dgarijo
Copy link
Owner

dgarijo commented Dec 11, 2023

@vChavezB thanks! Can you please merge in develop It's the branch where the latest changes are

@vChavezB vChavezB changed the base branch from master to develop December 11, 2023 14:40
@vChavezB
Copy link
Contributor Author

I have changed it to the develop branch. Should I also delete the webvowl patched zip and display the source files ? At the moment I tested my changes only with the resources from the folder loder and oops.

@dgarijo
Copy link
Owner

dgarijo commented Dec 11, 2023

We should be consistent, so yes, if it's not a lot of effort, let's do that. Having to create the zip file every time is less ideal.

@vChavezB vChavezB force-pushed the copy-direct-resources branch from fed3e06 to ba89c6b Compare December 11, 2023 15:09
@vChavezB
Copy link
Contributor Author

updated 👍

@dgarijo
Copy link
Owner

dgarijo commented Dec 12, 2023

This looks good to me. I'll have to check if those getAbsolutePaths do are not problematic in unix.
Thanks! Please add yourself as well in the contributor list in the README file

@dgarijo
Copy link
Owner

dgarijo commented Dec 12, 2023

Ouch, no, it does not pass the tests. I will leave a few comments where the errors may be

@vChavezB
Copy link
Contributor Author

Thanks for the feedback. I think I did not push the changes for those files correctly. Will have a look again ;)

@vChavezB vChavezB force-pushed the copy-direct-resources branch from ba89c6b to a365dbd Compare December 12, 2023 17:53
@dgarijo
Copy link
Owner

dgarijo commented Dec 12, 2023

Still not :(

@vChavezB
Copy link
Contributor Author

Sorry for the multiple changes. I must admit that Java is not my first language.

I will do a clean git clone and compile again to debug this.

@dgarijo
Copy link
Owner

dgarijo commented Dec 12, 2023

No rush

Instead of using zip files, use the source files in the
resources directory. This allows to track differences
easily over git as zip files cannot be compared
between commits easily.
@vChavezB vChavezB force-pushed the copy-direct-resources branch from a365dbd to 3ee9c26 Compare December 12, 2023 21:39
@vChavezB
Copy link
Contributor Author

Ok I think it should now compile, for some reason the previous commit did not use the copyResourceDir method.

@dgarijo
Copy link
Owner

dgarijo commented Dec 13, 2023

Thanks, looking good. Will review and merge soon!

@dgarijo dgarijo merged commit 9bbbbd3 into dgarijo:develop Dec 18, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants