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.

Sign up for GitHub

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

Jump to bottom

Lopp add checks #39

Closed
wants to merge 7 commits into from
Closed

Lopp add checks #39

wants to merge 7 commits into from

Conversation

slopp
Copy link
Collaborator

@slopp slopp commented Sep 1, 2023 •
edited
Loading

@github-actions
Copy link

github-actions bot commented Sep 1, 2023 •
edited
Loading

Your pull request is automatically being deployed to Dagster Cloud.

Location Status Link Updated
data-eng-pipeline View in Cloud Sep 15, 2023 at 08:28 PM (UTC)
basics View in Cloud Sep 15, 2023 at 08:28 PM (UTC)

@slopp slopp requested a review from izzye84 September 14, 2023 21:02
slopp and others added 3 commits September 14, 2023 16:23
@slopp
fix basics and avg_order numpy boolean issue
954dd8c
@izzye84
Update setup.py
871c571 
beautifulsoup4 required for the country_stats asset
@izzye84
Revert "Update setup.py"
086bb5b 
This reverts commit 871c571.
@slopp
Copy link
Collaborator Author

slopp commented Oct 9, 2023

Implemented in another PR

@slopp slopp closed this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@izzye84 izzye84 Awaiting requested review from izzye84

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@slopp @izzye84