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

Replace data package with /external/ git submodules #247

Merged
merged 3 commits into from
Jan 6, 2019

Conversation

behnam
Copy link
Member

@behnam behnam commented Jan 6, 2019

As we have now git repositories for the published data sources used in UNIC, we can replace our custom download package with git submodules, which are easier to maintain and provide more flexibility in dev time.


This change is Reviewable

Copy link
Member Author

@behnam behnam left a comment

Choose a reason for hiding this comment

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

bors: r+

Reviewed 88 of 88 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

bors bot added a commit that referenced this pull request Jan 6, 2019
247: Replace data package with /external/ git submodules r=behnam a=behnam

As we have now git repositories for the published data sources used in UNIC, we can replace our custom download package with git submodules, which are easier to maintain and provide more flexibility in dev time.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/open-i18n/rust-unic/247)
<!-- Reviewable:end -->


Co-authored-by: Behnam Esfahbod <behnam@zwnj.org>
@behnam behnam added this to the UNIC-0.9 milestone Jan 6, 2019
@bors
Copy link
Contributor

bors bot commented Jan 6, 2019

@bors bors bot merged commit 5b821df into open-i18n:master Jan 6, 2019
@behnam behnam deleted the data-1 branch January 6, 2019 15:35
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.

1 participant