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

Fix/40 error unable to create magasin dagster #52

Merged
merged 18 commits into from
Feb 9, 2024

Conversation

merlos
Copy link
Member

@merlos merlos commented Feb 9, 2024

@merlos merlos marked this pull request as draft February 9, 2024 07:52
@merlos
Copy link
Member Author

merlos commented Feb 9, 2024

The issue was that .gitignore included a line with instance/ and dagster helm chart includes a folder called templates/helpers/instance/. That folder was being ignored and therefore not included in the released version

@merlos merlos marked this pull request as ready for review February 9, 2024 09:46
@merlos merlos merged commit 004343c into main Feb 9, 2024
2 checks passed
@merlos merlos deleted the fix/40-error-unable-to-create-magasin-dagster branch March 1, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant