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

DO NOT MERGE: Repository consolidation testing #293

Closed
wants to merge 3 commits into from

Conversation

bmtcril
Copy link
Contributor

@bmtcril bmtcril commented Oct 24, 2024

I'm playing with the structure of how we can get all of the pieces to play nicely if the ADR to consolidate passes.

  • Move xapi-db-load (see commit notes)
  • Move aspects-dbt (see commit notes)
  • Move tutor-contrib-aspects code
  • Update docs

Adds the contents of the xapi-db-load repository as of commit
a1cd85603b451e02248793273764c38613c602ed . All make targets, including
tests and docs, should work. The xapi-db-load docs still need to be
moved and integrated with the overall Aspects docs site.
Moves the contents of aspects-dbt to this repository. There are a couple
of current issues with this:

- Old requirements for our clickhouse dbt stack conflict with the rest
of our constraints
- dbt docs generation / hosting won't work until we change the github
pages for it
@bmtcril bmtcril force-pushed the bmtcril/repo_consolidation branch 3 times, most recently from 70bcaf4 to 944dd2e Compare October 25, 2024 13:43
Bug fixes for CI around the new requirements versions and file
locations.
@bmtcril bmtcril force-pushed the bmtcril/repo_consolidation branch from 944dd2e to 6cb7a79 Compare October 25, 2024 13:50
@bmtcril bmtcril closed this Oct 25, 2024
@bmtcril bmtcril deleted the bmtcril/repo_consolidation branch October 25, 2024 15:23
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