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

[CI/CD Improvements] Migrate to hatch #1165

Closed
17 tasks
mikealfare opened this issue Aug 8, 2024 · 1 comment
Closed
17 tasks

[CI/CD Improvements] Migrate to hatch #1165

mikealfare opened this issue Aug 8, 2024 · 1 comment

Comments

@mikealfare
Copy link
Contributor

mikealfare commented Aug 8, 2024

Is this your first time submitting a feature request?

  • I have read the expectations for open source contributors
  • I have searched the existing issues, and I could not find an existing issue for this feature
  • I am requesting a straightforward extension of existing dbt-snowflake functionality, rather than a Big Idea better suited to a discussion

Describe the feature

As a maintainer, I want to standardize and modernize development tooling, so that I can reduce maintenance issues.

As a maintainer, I want to automate formatting checks, so that I can focus more on content in PRs and still maintain a code quality standard.

As a maintainer, I want to simplify the CI jobs, so that I know I am running the same CI regardless of scenario (e.g. PR vs scheduled tests vs end to end tests for dependent packages).

As a maintainer, I want to simplify the CD jobs, so that deployment is more reliable and stable.

Acceptance Criteria

Preview Give feedback

Suggestions

  • Use dbt-postgres as a template
  • It's fine to copy paste; it's worth violating DRY for the sake of isolation in a migration of this size

Style standards:

  • Job ids, step ids, and variables are in kebab case
  • Job names, step names, and description fields follow dbt docs standards (capitalize first word only)
  • Extra descriptors should be avoided unless required for disambiguation, e.g.
    • version-number -> version
    • archive-name -> archive
  • Workflow files use a four space tab
  • Scripts (inline or separate files) use environment variables in env instead of inline substitution like ${{ inputs.value }}
@mikealfare
Copy link
Contributor Author

This issue is consolidated into #1004.

@mikealfare mikealfare closed this as not planned Won't fix, can't repro, duplicate, stale Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant