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

Allow to alter link fields of kind many #49

Merged
merged 7 commits into from
Feb 7, 2025
Merged

Allow to alter link fields of kind many #49

merged 7 commits into from
Feb 7, 2025

Conversation

colodenn
Copy link
Member

@colodenn colodenn commented Feb 7, 2025

This update enables alteration to fields of type link with kind: 'many'. Additionally, I've reorganized the tests for improved clarity and eliminated much of the redundant code. I also introduced helper functions to format SQL statements, making them more readable.

@colodenn colodenn added bug Something isn't working enhancement New feature or request labels Feb 7, 2025
@colodenn colodenn requested a review from leo February 7, 2025 14:55
@colodenn colodenn self-assigned this Feb 7, 2025
Copy link

github-actions bot commented Feb 7, 2025

Released an experimental package:

bun add @ronin/cli@corny-ron-1099-experimental-118

This package will be removed after the pull request has been merged.

@codecov-commenter
Copy link

codecov-commenter commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
src/utils/field.ts 100.00% <100.00%> (ø)
src/utils/format.ts 100.00% <100.00%> (ø)
tests/fixtures/index.ts 100.00% <100.00%> (ø)
tests/fixtures/utils.ts 100.00% <100.00%> (ø)

@colodenn colodenn marked this pull request as ready for review February 7, 2025 15:03
leo
leo previously requested changes Feb 7, 2025
leo
leo previously requested changes Feb 7, 2025
Co-authored-by: Leo Lamprecht <[email protected]>
Copy link
Member

@leo leo left a comment

Choose a reason for hiding this comment

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

Very nice!

@colodenn colodenn merged commit 15ee4e6 into main Feb 7, 2025
3 checks passed
@colodenn colodenn deleted the corny/ron-1099 branch February 7, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants