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

Add test #47

Merged
merged 4 commits into from
Oct 20, 2022
Merged

Add test #47

merged 4 commits into from
Oct 20, 2022

Conversation

bertsky
Copy link
Member

@bertsky bertsky commented Oct 20, 2022

No description provided.

@bertsky bertsky requested a review from markusweigelt October 20, 2022 13:17
@bertsky bertsky linked an issue Oct 20, 2022 that may be closed by this pull request
README.md Outdated Show resolved Hide resolved
README.md Outdated
make prepare
make prepare # generate all missing files
make prepare-keys # generate only SSH credentials
make prepare-data # generate only Kitodo OCR scripts
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
make prepare-data # generate only Kitodo OCR scripts
make prepare-data # generate data which are required for all profiles

Copy link
Member Author

Choose a reason for hiding this comment

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

why all profiles? It's only Kitodo, isn't it? Also, it's just the scripts I thought...

Copy link
Collaborator

@markusweigelt markusweigelt Oct 20, 2022

Choose a reason for hiding this comment

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

Yes. Atm there are just the scripts but it is conceivable that further Kitodo files and folders will be added or service folders. We can change if its necessary.

README.md Outdated
make prepare # generate all missing files
make prepare-keys # generate only SSH credentials
make prepare-data # generate only Kitodo OCR scripts
make prepare-examples # generate only Kitodo DB entries
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
make prepare-examples # generate only Kitodo DB entries
make prepare-examples # generate example data for Kitodo.Production for example DB entries and process

Copy link
Member Author

Choose a reason for hiding this comment

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

we already have examples in the name of the target – no need to repeat twice. (Also, the explanation above already enumerates which things are added to the database.)

@markusweigelt markusweigelt merged commit fd5f3af into slub:main Oct 20, 2022
@bertsky bertsky deleted the add-test branch December 14, 2023 12:26
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.

add regression test
2 participants