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

Changes some columns #64

Closed
7 tasks
HamiltonRepoMigrationBot opened this issue Feb 26, 2023 · 1 comment
Closed
7 tasks

Changes some columns #64

HamiltonRepoMigrationBot opened this issue Feb 26, 2023 · 1 comment
Labels
migrated-from-old-repo Migrated from old repository

Comments

@HamiltonRepoMigrationBot
Copy link
Collaborator

Issue by elijahbenizzy
Friday Jan 27, 2023 at 20:36 GMT
Originally opened as stitchfix/hamilton#287


[Short description explaining the high-level reason for the pull request]

A few minor changes -- just playing around with this. Intent is not to merge.

Changes

How I tested this

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

elijahbenizzy included the following code: https://github.com/stitchfix/hamilton/pull/287/commits

@elijahbenizzy elijahbenizzy added the migrated-from-old-repo Migrated from old repository label Feb 26, 2023
@elijahbenizzy
Copy link
Collaborator

Old PR, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migrated-from-old-repo Migrated from old repository
Projects
None yet
Development

No branches or pull requests

2 participants