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 missing language tag #663

Merged
merged 1 commit into from
Jan 6, 2025
Merged

Add missing language tag #663

merged 1 commit into from
Jan 6, 2025

Conversation

acka47
Copy link
Contributor

@acka47 acka47 commented Jan 6, 2025

..when automatically creating the classification to avoid errors in SkoHub shape validation process.

See failing test at https://github.com/hbz/lobid-vocabs/actions/runs/12633659256/job/35199950424 and me manually adding the tag in hbz/lobid-vocabs@f271dc5 (and also hbz/lobid-vocabs@79b0bad)

@acka47 acka47 requested a review from fsteeg January 6, 2025 14:01
@acka47
Copy link
Contributor Author

acka47 commented Jan 6, 2025

Hmm, I do not see why the checks fail and am not sure whether it actually has anything to do with my change.

@fsteeg
Copy link
Member

fsteeg commented Jan 6, 2025

You probably didn't branch off the current main branch (which contains the fix for the build error).

..when automatically creating the classification to avoid errors in
SkoHub shape validation process.
@acka47 acka47 force-pushed the addMissingLanguageTag branch from 05fd946 to e615b5d Compare January 6, 2025 14:27
@acka47
Copy link
Contributor Author

acka47 commented Jan 6, 2025

You probably didn't branch off the current main branch (which contains the fix for the build error).

Thanks, I pulled the changes and force-pushed (sorry) into this branch.

@fsteeg fsteeg merged commit d830441 into master Jan 6, 2025
1 check passed
@fsteeg fsteeg deleted the addMissingLanguageTag branch January 6, 2025 14:46
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