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

[du-dbt] adding starter project to monorepo #20162

Merged

Conversation

tacastillo
Copy link
Contributor

@tacastillo tacastillo commented Feb 29, 2024

Summary & Motivation

I think I deleted all of the local files that should be ignored, but Erin, can you double check?

How I Tested These Changes

@tacastillo
Copy link
Contributor Author

tacastillo commented Feb 29, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tacastillo and the rest of your teammates on Graphite Graphite

@yuhan
Copy link
Contributor

yuhan commented Mar 2, 2024

should we have a root dir level gitignore and include .env?

@tacastillo tacastillo force-pushed the 02-29-_du-dbt_adding_starter_project_to_monorepo branch 6 times, most recently from e32d3bf to aad0cf7 Compare March 6, 2024 19:41
@tacastillo
Copy link
Contributor Author

tacastillo commented Mar 6, 2024

Merge activity

@tacastillo tacastillo merged commit 588b7f9 into master Mar 6, 2024
1 check passed
@tacastillo tacastillo deleted the 02-29-_du-dbt_adding_starter_project_to_monorepo branch March 6, 2024 21:59
dpeng817 pushed a commit that referenced this pull request Mar 6, 2024
## Summary & Motivation

I think I deleted all of the local files that should be ignored, but Erin, can you double check?

## How I Tested These Changes
dpeng817 pushed a commit that referenced this pull request Mar 6, 2024
## Summary & Motivation

I think I deleted all of the local files that should be ignored, but Erin, can you double check?

## How I Tested These Changes
PedramNavid pushed a commit that referenced this pull request Mar 28, 2024
## Summary & Motivation

I think I deleted all of the local files that should be ignored, but Erin, can you double check?

## How I Tested These Changes
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