Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
retoor9881d@Lensflare that you're willing to write a whole SDK while the platform isn't that popular anymore proves you're a true dev that devs for fun as it should! I hate it when people ask "Why did you do that.". I'ts fun.
Less online than a hour and you have in depth review: https://retoor.molodetz.nl/retoor/...
devRant isn't the biggest community anymore, but still has high engagement among users!
Myself is quite happy with the review quality. It's better than in human when it comes to pure technical stuff (it doesn't do business logic ofc). You have almost an eight which is very high. Only one other friend has that and i didn't do at all on first check. My projects often have high complexity and have a lot to bash on. So you think you made some master code and then this system says, that's worth a six!
My review system is perfect to reflect yourself if working alone.
Now I have done this, I will read the source myself too but I have no experience with swift. -
@retoor Thank you!
Cool stuff, your automatic review system.
It picked up some good points but at some places it fails to understand that what it criticizes is actually intentional. :)
For example the differences in property naming of the CodingData models and the "main" models is intentional because the sdk tries to map the devrant structure to something that is easier to use and understand.
Devrant often uses cryptic names like b for background color and a for avatar image.
The sdk also maps Integer booleans to actual booleans and integer dates to actual dates, etc.
The user mapping caching is a valid point. I thought about it myself and I‘m impressed that it picked that up.
Valid, but probably unnecessary unless the data set is huge, which it isn‘t.
The request that gets the data is orders of magnitude slower than it takes to map that data π -
retoor9881d@Lensflare what about the opinions of files that got graded a 4?
Yeah, what i said, in technical sense it beats a human. But things like naming because a remote service uses that name is something a human understands but AI doesn't.
I would be professionally fine with :
- review.
- automated review.
- no review at all, pins in names, but writing tests is required and of course manual testing.
I am not a fan of reviews made by humans normally because they're not that good. Human linters sometime. Also, reviewing is often harder than coding if you do it well and lets face it, we spend a lot less effort on someones code to review than on our own code. Also, at a certain moment you know eachother's strengths, weaknesses and you kinda believe from eachother that it's coded well.
Tests are required, but review process can be ditched unless asked for. I know, unpopular opinion. Maybe just voluntary reviews instead of required ones is idea. Don't review my button text. -
retoor9881d@Lensflare I also added positive points to the review and I like it a lot! It's nice if certain stuff that you do effort for is notices! This thing is more social than many coworkers! It also remembers your birthday btw.
-
retoor9881d@Lensflare fuck, the css doesn't show link on my site. But the DevRantSDK is clickable and you got directory with review per file in project structure. Can be that site will be a few times down, I have to reboot the platform (it's not just a site) to apply changes because of caching. It generates pages in less than a few Ms for a reason. Reason 1: barely any JS. Especially not for content.
-
retoor98824h@Lensflare It's in progress to make nice markdown reviews of it, but you have a lot to read: https://retoor.molodetz.nl/retoor/...
:P -
retoor98824h@Lensflare the review system is really doing great work, it looks good. but the first digit you see on pages is the grade that specific file has: https://retoor.molodetz.nl/retoor/...
You have a few with four, but have no idea which files they are :P -
retoor98824h@Lensflare found a bug https://retoor.molodetz.nl/retoor/...
The system should have a complete summary of bugs on the front page tbh. Should write that. -
retoor98823h@Lensflare Yh, maybe it's a bit outdated. The autocompletion AI does for C# is also outdated, my vscode corrects it all with the right way to go.
-
retoor9885h@Lensflare, you got me convinced and I will write a project in swift. But sadly, your poject doesn't compile out of the box. There are library issues between linux and macOs. Linux uses the FoundationNetworking module. A different one than you do. I will try to repair it and make it work. Maybe i'll make a bot with it, a bit more decent than with python.
Btw, wtf, why does it it build to a ".build" directory. Yh, I can guess, for the .gitignore. But c'mon. Very unexpected.
I did read the basic tutorial 50% tho and that's how I could convinced of the language. Swift exactly took over what I like from different languages and kept the good shit. This is really an innovate language without having bullshit like Rust has. I am familiar with many languages it's based on I guess. Would be awesome to work with. New invention. I'm so grateful.
You project did lead to maybe a new swiftii dev, is there a better compliment? -
retoor9885h@Lensflare fuck, it's in the KreeRequest module. Oh oh, nothing to be found icm with Linux. This is a short love story, will probably make smth else than a chatbot with swift then. Replacing the KreeRequest module is the half project or so.
-
@retoor glad to hear that! π
Swift really takes inspiration from many other languages and keeps the good parts.
KreeRequest is a really thin layer on top of the stdlib URLSession.
I wasn‘t aware that it‘s not available for linux.
I‘m interested to provide support for linux so I will take a look at it and make it possible.
Related Rants
I have finished my project for a new devrant sdk in swift:
https://github.com/WilhelmOks/...
(Yes, the readme is missing and I will add it later π )
And I built it into JoyRant (build 32), which required quite a bit of refactoring.
I think I tested everything but let me know if you find any problems.
You will need to logout and log in again.
The downvote reason feature isn‘t there yet (in JoyRant), but now I have a good basis to add it π
Yes, implementing a whole new SDK just for one little feature is overkill, but I had fun doing it and I wanted to have a nice, clean and extensible code base which I can work on in the future. π
devrant
joyrant