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

Switch from submodule "vocabs" to "lbz-vocabs" #2064

Merged
merged 5 commits into from
Sep 5, 2024

Conversation

dr0i
Copy link
Member

@dr0i dr0i commented Aug 29, 2024

We now use a dedicated vocabs repo for RPB.

  • add git submodule "lbz-vocabs"
  • improve README
  • configure start script

See #2063.

We now use a dedicated vocabs repo for RPB.

- add git submodule "lbz-vocabs"
- improve README
- configure start script
@dr0i dr0i requested a review from blackwinter August 29, 2024 11:43
@dr0i dr0i added the ALMA label Aug 29, 2024
@dr0i dr0i requested review from fsteeg and removed request for blackwinter August 29, 2024 11:46
@dr0i dr0i assigned fsteeg and unassigned blackwinter Aug 29, 2024
fixVariables.put("rpb2.ttl", "../../../../../../vocabs/rpb/rpb2.ttl");
fixVariables.put("rpb-spatial.ttl", "../../../../../../vocabs/rpb/rpb-spatial.ttl");
fixVariables.put("rpb.ttl", "../../../../../../vocabs/rpb/rpb.ttl");
fixVariables.put("rpb2.ttl", "../../../../../../lbz-vocabs/rpb/rpb2.ttl");
Copy link
Contributor

@acka47 acka47 Aug 29, 2024

Choose a reason for hiding this comment

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

This file will finally most probably be renamed to lbz-notationen.ttl, see hbz/lbz-vocabs@38b3e89

Copy link
Contributor

Choose a reason for hiding this comment

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

This means, we should wait with mergin this until hbz/lbz-vocabs#6 is merged and adjust the file name if necessary.

Copy link
Contributor

Choose a reason for hiding this comment

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

I have made a separate PR for changing the file name at hbz/lbz-vocabs#7 so that we can move on with this PR. @TobiasNx will review over there.

@fsteeg fsteeg removed their assignment Aug 29, 2024
@fsteeg fsteeg removed their request for review August 29, 2024 15:21
@TobiasNx
Copy link
Contributor

TobiasNx commented Sep 3, 2024

I also adjusted the filenames and mappings.

@acka47 acka47 self-requested a review September 4, 2024 06:58
Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

+1 This looks good. It can and should be merged now as the lbz vocabs are now removed from lobid-vocabs with hbz/lobid-vocabs#230

@dr0i
Copy link
Member Author

dr0i commented Sep 5, 2024

@acka, you wrote:

It can and should be merged now

This PR is asigned to you still. Who is going to merge this? Would you assign me, if that is what you want?

@acka47 acka47 assigned dr0i and unassigned acka47 Sep 5, 2024
fixVariables.put("rpb.ttl", "../../../../../../vocabs/rpb/rpb.ttl");
fixVariables.put("lbz-notationen.ttl", "../../../../../../lbz-vocabs/rpb/lbz-notationen.ttl");
fixVariables.put("rpb-spatial.ttl", "../../../../../../lbz-vocabs/rpb/rpb-spatial.ttl");
fixVariables.put("rpb.ttl", "../../../../../../lbz-vocabs/rpb/rpb.ttl");
Copy link
Member Author

Choose a reason for hiding this comment

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

There is no rpb directory in https://github.com/hbz/lbz-vocabs . I guess that rpb-subdirectory has just to be removed here?

Copy link
Contributor

Choose a reason for hiding this comment

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

you are right!!! thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Done!

@dr0i dr0i assigned TobiasNx and unassigned dr0i Sep 5, 2024
@TobiasNx TobiasNx assigned dr0i and unassigned TobiasNx Sep 5, 2024
@dr0i dr0i merged commit 1f100be into master Sep 5, 2024
1 check passed
@dr0i dr0i deleted the 2063-configureLbzVocabsSubmodule branch September 5, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants