3 ms·
Someone's given it a shot: https://github.com/obsidianmd/obsidian-importer/pull/424 https://github.com/obsidianmd/obsidian-importer/pull/424
by Banditoz 1y ago
Someone's given it a shot: https://github.com/obsidianmd/obsidian-importer/pull/424 https://github.com/obsidianmd/obsidian-importer/pull/424
- sarreph 1y agoCan't help but think if the author of that PR had been less defeatist and snarky they would have had a chance at decent discussion about it being a viable option (with AI).
- jychang 1y agoIt's 1100 lines of code in a single file that nobody understands. That's instant spaghetti right there, not a valid PR. At least have it be split into some files and structured in some way.
- artursapek 1y agohow is a single 1100 line file that handles Notion’s API format “spaghetti”? spaghetti code is code that snakes through the code base in a hard-to-follow way
- ath3nd 1y ago[dead]
- sndixnnx 1y agoIt’s funny reading these threads. A gif of a few clicks is evidence “it works” for the author. It’s like citing lines of code as a measure of your productivity. Appears impressive on first glance, but any expert will call you out as a snake oil salesman. My guess is this is close to the level of testing they put forth for ensuring the AI generated code works (based off my experience with other AI heavy devs). They didn’t take any time to thoroughly review nor understand the code. A large file doesn’t necessarily mean shoddy work, but it certainly indicates it’s likely that.