Skip to content

Move ZAPDTR and OTRExporter to sub modules#3298

Merged
louist103 merged 12 commits intoHarbourMasters:developfrom
louist103:ZAPD_TEST_101623
Oct 17, 2023
Merged

Move ZAPDTR and OTRExporter to sub modules#3298
louist103 merged 12 commits intoHarbourMasters:developfrom
louist103:ZAPD_TEST_101623

Conversation

@louist103
Copy link
Contributor

@louist103 louist103 commented Oct 17, 2023

subrepo:
  subdir:   "ZAPDTR"
  merged:   "a05b4b9e"
upstream:
  origin:   "none"
  branch:   "master"
  commit:   "none"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"
subrepo:
  subdir:   "ZAPDTR"
  merged:   "e6b07647"
upstream:
  origin:   "https://github.com/harbourmasters/ZAPDTR"
  branch:   "TEST_101623"
  commit:   "e6b07647"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"
subrepo:
  subdir:   ".\OTRExporter\"
  merged:   "b71a2506"
upstream:
  origin:   "none"
  branch:   "master"
  commit:   "none"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"
subrepo:
  subdir:   "OTRExporter"
  merged:   "89de1472"
upstream:
  origin:   "https://github.com/harbourmasters/OTRExporter"
  branch:   "TEST-101623"
  commit:   "89de1472"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"
Copy link
Contributor

@briaguya0 briaguya0 left a comment

Choose a reason for hiding this comment

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

tested locally on linux, builds and runs fine with submodule init/update

Copy link
Contributor

@garrettjoecox garrettjoecox left a comment

Choose a reason for hiding this comment

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

Building for me on mac as well

@louist103
Copy link
Contributor Author

Working on Windows.

@louist103 louist103 merged commit 4a70fea into HarbourMasters:develop Oct 17, 2023
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.

3 participants