Skip to content

show guids properly in MS SQL #174

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

Merged

Conversation

kmiterror
Copy link
Contributor

Guids are not shown properly in the table
This fixes it
Before
image

After
image

@kmiterror kmiterror force-pushed the fix-scrambled-guids-in-ms-sql-db branch from 46f6d09 to 4e2fef5 Compare March 14, 2025 13:53
@jorgerojas26
Copy link
Owner

Hahahaha this is funny, don't let the ci win the fight.

@kmiterror
Copy link
Contributor Author

Hahahaha this is funny, don't let the ci win the fight.

Sorry for the chaos
I have no idea what am I doing
I started with go like yesterday
Just added some lsp to my neovim setup, maybe this will make things easier
Stay tuned for more fun :D

@jorgerojas26
Copy link
Owner

Hahahaha this is funny, don't let the ci win the fight.

Sorry for the chaos I have no idea what am I doing I started with go like yesterday Just added some lsp to my neovim setup, maybe this will make things easier Stay tuned for more fun :D

That's okay, i'm not experienced in Go either.

You can run the linter locally with the command "golangci-lint run".

@kmiterror kmiterror force-pushed the fix-scrambled-guids-in-ms-sql-db branch from d797303 to 040c1bb Compare March 18, 2025 14:47
kmiterror and others added 2 commits March 18, 2025 21:19
Copy link
Contributor

@ccoVeille ccoVeille left a comment

Choose a reason for hiding this comment

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

LGTM 👍, except a typo (I made)

Co-authored-by: ccoVeille <[email protected]>
@jorgerojas26 jorgerojas26 merged commit 423124f into jorgerojas26:main Mar 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants