Skip to content

Comments

feat: ✨ update package properties#13

Merged
lwjohnst86 merged 7 commits intomainfrom
feat/update-package-properties
Jan 19, 2026
Merged

feat: ✨ update package properties#13
lwjohnst86 merged 7 commits intomainfrom
feat/update-package-properties

Conversation

@K-Beicher
Copy link
Contributor

@K-Beicher K-Beicher commented Jan 15, 2026

Description

Updated the information in the package, there is a problem with my just file, so had to add an exemption to the description. I'm working on fixing that.

Closes #5

This PR needs a quick review.

Checklist

  • Ran just run-all

@K-Beicher K-Beicher self-assigned this Jan 15, 2026
@K-Beicher K-Beicher linked an issue Jan 15, 2026 that may be closed by this pull request
@K-Beicher K-Beicher moved this from Todo to In Review in Data Activity Jan 15, 2026
@lwjohnst86 lwjohnst86 changed the title Feat: update package properties feat: ✨ update package properties Jan 15, 2026
Copy link
Collaborator

@martonvago martonvago left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, just a comment about comments ⭐

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Data Activity Jan 15, 2026
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good start! Some comments

Co-authored-by: Luke W. Johnston <lwjohnst86@users.noreply.github.com>
Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
@lwjohnst86 lwjohnst86 removed the status in Data Activity Jan 16, 2026
@lwjohnst86 lwjohnst86 moved this to In Review in Data Activity Jan 16, 2026
Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍 some small comments

@github-project-automation github-project-automation bot moved this from In Review to In Progress in Data Activity Jan 16, 2026
@K-Beicher K-Beicher requested a review from lwjohnst86 January 19, 2026 09:39
@K-Beicher K-Beicher moved this from In Progress to In Review in Data Activity Jan 19, 2026
@K-Beicher
Copy link
Contributor Author

@lwjohnst86 Was it supposed to make that many updates to the uv.lock file once I'd updated the uv installation?

Copy link
Member

@lwjohnst86 lwjohnst86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea thats fine 👍 great!

@lwjohnst86 lwjohnst86 merged commit 7101392 into main Jan 19, 2026
@lwjohnst86 lwjohnst86 deleted the feat/update-package-properties branch January 19, 2026 09:52
@github-project-automation github-project-automation bot moved this from In Review to Done in Data Activity Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Fill in package properties

3 participants