HNHacker News
TopNewBestAskShowJobs

ivankahl

121 karma · joined September 3, 2018

submissionscomments
ivankahl··on Ask HN: How to deal with long vibe-coded PRs?
What are your organization's expectations or policies regarding PR size and acceptable AI usage? Even if your organization hasn't set any expectations, what are yours—and have you communicated them to the author?

If expectations have been shared and these changes contradict them, you can quickly close the PR, explain why it's not acceptable, and ask them to redo it.

If you don't have clear guidelines on AI usage or haven't shared your expectations, you'll need to review the PR more carefully. First, verify whether your assumption that it’s a simple service is accurate (although from your description, it sounds like it is). If it is, talk to the author and point out that it's more complicated than necessary. You can also ask if they used AI and warn them about the complexities it can introduce.

ivankahl··on What's New in C# 14: Null-Conditional Assignments
Thanks for the upvotes! While testing and writing about the feature, I suspected it would receive mixed reactions.

The `?.` operator behaves similarly on the LHS to the RHS, making the language more consistent, which is always a good thing. In terms of readability, I would say that once you understand how the operator works (which is intuitive because the language already supports it on the RHS), it becomes more readable than wrapping conditionals in `if` statements.

There are downsides, such as the overuse I mentioned. But this is true for many other language features: it requires experience to know when to use a feature appropriately, rather than applying it everywhere.

However, the great thing about this particular enhancement is that it's mostly cosmetic. Nothing prevents teams from not adopting it; the old syntax still works and can be enforced. C# and .NET are incredibly versatile, which means code can look dramatically different depending on its context and domain. For some projects, this feature might not be needed at all. But many codebases do end up with various conditional assignments, and in those cases, this can be useful.

ivankahl··on Grammarly acquires Superhuman
Agree with this. I love that Grammarly is integrated into everything, and I don't have to switch to an AI chat for minor text edits.
ivankahl··on Show HN: Embed Draw.io in Notion
Interesting will definitely check this out
ivankahl··on Show HN: Embed Draw.io in Notion
Hi guys,

I recently wanted to embed a draw.io diagram in Notion but couldn’t get it to work with the default Export as URL option in draw.io as it doesn’t load correctly in Notion.

This led me to build this super simple HTML page that I can use to specify the draw.io link and then get a link that can be embedded in Notion.

So far it’s worked really well for me so I thought I’d polish it up and share it with everyone in case you maybe find it useful!

Thanks!