reviewboard, to random
@reviewboard@mastodon.online avatar
doctormo, to python
@doctormo@floss.social avatar

I'm slowly making my way through a 8 thousand line python addition to 's library for parsing text and font elements in properly. It's important and useful, but it's also a lot.

I'm not sure how to say "please run pylint over your code because my eyes hurt trying to read this". without sounding like an ungrateful jerk. 😅

develwithoutacause, to javascript
@develwithoutacause@techhub.social avatar

: Always use === over ==.

=== applies much more reasonable behavior for operands of different types, mainly by not coercing them together like == does.

A lot of developers will tell you to learn the rules of coercion and use it when appropriate, however I disagree for one key reason. Consider this example:

if (foo == bar) {  
 doSomething();  
}  

Question: Did the developer mean to use ==? Is the coercion intended or a typo?

It's incredibly difficult to know with any amount of certainty as this depends on the types and semantics of foo and bar.

If I was writing this intentionally, I would feel compelled to write a several line comment about how coercion behavior applies here in a desirable way. And if you need to write that much explanation, it would be much less confusing to actually codify the desired behavior with === and explicit type checks so devs don't have to understand that coercion.

morenonatural, to github Spanish
@morenonatural@todon.nl avatar

[2208.04259] First Come First Served: The Impact of File Position on #CodeReview
https://arxiv.org/abs/2208.04259

#github #gerrit #gitlab #sourceforge

weddige, (edited ) to foss
@weddige@gruene.social avatar

#FOSS offers companies a good deal: Great, cheap software that can be used in all sorts of products and projects. But of course you can't just outsource the responsibility because it's maintained by some random person in Nebraska.

So I have a question for software #developers working in small and medium sized companies (big companies can pay themselves AND leave money for maintainers):

Would your company be interested in crowd-funded #codereview​s of the FOSS projects you are using?

chipx86, to github
@chipx86@mastodon.online avatar

I just realized, before I know it, we'll be hitting 20 years of @reviewboard Man, do I feel old.

It's hard to imagine it now, but #codereview wasn't really a thing when we built this. There were a couple expensive enterprise tools, but #GitHub? Pull requests? They didn't exist.

We had to solve a lot of problems that didn't have readily-made solutions, like:

FakeScrumStats, to memes
@FakeScrumStats@techhub.social avatar
reviewboard, to random
@reviewboard@mastodon.online avatar

💡 Tip: Use Review Board's image and document review to review the other parts of your project. We'll even show you diffs of your #PDF documents and presentations!

#codereview #docreview

jonny, to random
@jonny@neuromatch.social avatar

Call for @joss reviewers:

EdgeVPN.io
repo: https://github.com/EdgeVPNio/evio
pre-review: https://github.com/openjournals/joss-reviews/issues/6355
language: Python

Description:

EdgeVPN.io is an evolution of the IP-over-P2P (IPOP) project. IPOP started as an IP-based peer-to-peer overlay targeting personal devices, and over time the architecture evolved to adopt various standards, support centralized user/group management, and incorporate software-defined networking, culminating in the current architecture, tailored for research and development in nascent edge computing applications.
...
EdgeVPNio is a research project to build networking for the fog, spanning the network continuum from the cloud to its edge. It builds networking cyber-infrastructure which supports emerging IoT era applications.

Looks like this one might be a bit of fun for people, or i suppose people generally. No prior experience reviewing for JOSS is required, experience with Python is required, and some experience with the topic area is preferred. Don't be shy! If you've never done open review before, JOSS is a great place to start. It's a really good way to learn by teaching (or learn by reading!) in a collaborative context. You can reply here or on the pre-review issue to volunteer :)

edit: would love to have some infosec people on this one! even and especially if you are not in academia :)

drupalfr, to drupal French
@drupalfr@drupal.community avatar

Petit changement dans le programme : suite à l'indisponibilité d'une conférencière, nous avons proposé à un autre binôme de présenter un sujet sur la Revue de code !

Merci à Lydie FROMONT et Florent Torregrosa d'avoir accepté ce petit changement 😘

https://rennes2024.drupalcamp.fr/programme/conferences/revue-code/

A retrouver vendredi à 14h45 !

reviewboard, to linux
@reviewboard@mastodon.online avatar

We’ve just launched a new installer for Review Board, with support for over 50 different system environments.

We wanted to make it easy to get up-and-running quickly, instead of following a page of step-by-step instructions.

So now it's as simple as typing a single command:

$ curl https://install.reviewboard.org | python3

https://www.reviewboard.org/news/2024/03/05/installing-review-board-has-never-been-easier/

jonny, (edited ) to python
@jonny@neuromatch.social avatar

I'm looking for reviewers for two packages at the moment:

Automata (@pyOpenSci )
Review: https://github.com/pyOpenSci/software-submission/issues/152
Repo: https://github.com/caleb531/automata
A #Python library for simulating finite #automata, pushdown automata, and Turing machines.

Kirstine.jl
( @joss )
Review: https://github.com/openjournals/joss-reviews/issues/6193
Repo: https://sr.ht/~lsandig/Kirstine.jl
A #Julia package for Bayesian optimal experimental design with nonlinear regression models.

You'll be working with another reviewer to read and run the code, make sure it fills a basic checklist which usually only takes a few hours, and beyond that whatever youd like to focus on. Both of these are collaborative review processes where the goal is to help these packages be usable, well documented, and maintainable for the overall health of free scientific software.

Its fun, I promise! Happy to answer questions and boosts welcome.

Edit: feel free to volunteer as a reply here, DM me, or commenting on those issues! Anyone is welcome! Some experience with the language required, but other than that I can coach you through the rest.

#PeerReview #OpenReview #CodeReview #FiniteAutomata #TuringMachines #Bayesian #Regression #Statistics

schizanon, to webdev
@schizanon@mas.to avatar

This might be heresy but:

  1. Code reviews are a massive productivity tax with tiny quality benefits
  2. They should not be mandated
  3. The author should feel free to request a review if they want it
  4. If you don't trust your engineers, invest more in CI, or hire better ones

schizanon, to programming
@schizanon@mas.to avatar

If two programmers disagree in code review, one of them thinks a problem exists that doesn't, or vice versa.

People code for the problems they see. When you dismiss their solutions you dismiss their point of view.

You should never say "your solution is wrong." only "what problem are you solving?"

Problems are concrete, and you can't solve all of them at the same time. You can say "we aren't solving that problem, yet."

jonny, to random
@jonny@neuromatch.social avatar

Anyone up for a little light ? Glitch has an option to show boosts in the local TL, which I think is lovely for small instances like ours, it really helps new ppl get oriented in the fedi. The problem is public feeds dont filter duplicate boosts in the same way that home feeds do, so you end up seeing the same boost lots of time.

I wrote a patch to only show the most recent boost in public TLs, but since I am a relative noob with postgres, SQL, and rails, and since it touches core feed functionality id love it if someone took a look before we deploy on our instance (after some field testing we'll pull upstream). I tried to make it as contained as I could so other masto forks could merge it in too, if ya interested.

PR: https://github.com/NeuromatchAcademy/mastodon/pull/36
Wiki page with explanation: https://wiki.neuromatch.social/Filter_Duplicate_Boosts

reviewboard, to security
@reviewboard@mastodon.online avatar

New security/bug fix releases: Review Board 6.0.2, 5.0.7, 4.0.13, 3.0.26.

https://reviewboard.org/news/2024/01/16/review-board-security-bugfix-releases-6-0-2--5-0-7--4-0-13--3-0-26/

We discovered a vulnerability in-house allowing users with legitimate access to a server to craft a specific API request to fetch diff content they don't have permission to see.

FakeScrumStats, to random
@FakeScrumStats@techhub.social avatar
hnsec, to Blog

As we wrap up 2023, let's take a look back at the different topics we covered in our technical #blog this year.

Our #VulnerabilityResearch series expanded with some new writeups and coordinated disclosure advisories. We also provided practical advice and tooling to aid security researchers in effective #CodeReview using #Semgrep. There’s more in store on this topic: stay tuned for the latest updates.

Exploring various aspects of #OffensiveSecurity, we shared tools and methodologies for #RedTeaming, #WebPentesting, and #MobilePentesting. Don’t miss our popular series on customizing the #Sliver adversary emulation framework and extending #BurpSuite.

As we look forward to another year of research and community sharing, we wish you all happy holidays... and happy hacking!

https://security.humanativaspa.it/

reviewboard, to ai
@reviewboard@mastodon.online avatar

While #AI can be a powerful tool for software development and code review, we believe it must be integrated responsibly, with great care and appreciation for people's data, IP, and #privacy.

We strive to put people first. Future AI features will emphasize:

  • Full user control and opt-in
  • Transparency around data use
  • Enhancing, not replacing, human expertise

Read our AI Ethics Policy to learn more. It's our promise to you.

https://www.beanbaginc.com/ai-ethics-policy/

#software #development #codereview

pwntester, (edited ) to random

Discover the latest insights from our team’s audit on Home Assistant security! 🛡️ https://github.blog/2023-11-30-securing-our-home-labs-home-assistant-code-review/
#CodeReview

eldamir, to random
@eldamir@hachyderm.io avatar

I need a #codereview for last years #AdventOfCode Day 9.

I feel like I’ve tested everything, so there must be some edge case, I’m not seeing.

It is simple C#, ready to pull and run, with xunit tests in place. My personal input, and the task description is in there as well, so everything is readily available for anyone to pick up and chime in 😊

Any help or hints would be much appreciated 🙏❤️

https://github.com/eldamir/AdventOfCode2022/pull/1

raptor, to random

My #vulnerability research work on the #Zephyr RTOS is finally public on the @hnsec blog! 🚀

https://security.humanativaspa.it/ost2-zephyr-rtos-and-a-bunch-of-cves/

What better way to celebrate 20 years since my first CVE than with a bunch of brand new CVEs? 😉

Thanks to the @OpenSecurityTraining2 project for inspiring me to pursue this #CodeReview challenge. Cheers 🥂

raptor, to web

I’m really proud of the @hnsec blog!

We published articles about #web and #mobile #appsec, #IoT, #redteaming, #vulndev, #codereview

In a bit more than a couple of years of activity that’s a lot in my opinion. And we have more to come soon!

Well done, #hnsecurity team 💚

https://security.humanativaspa.it/

audiodude, to meta

Did you know that when you view a toot that originated on another server, you don't see all of the toots replies?

You will only see replies that would have, basically, shown up in your federated timeline (posted by someone whom someone on your server follows).

I created a PR on Sep 13 to address this and display an information message that allows you to easily click through to the original server to see all of the replies, much like with profiles etc.

https://github.com/mastodon/mastodon/pull/26910

#meta #mastodon

audiodude,

So far I've seen no comments, no discussion, nothing on my PR:

https://github.com/mastodon/mastodon/pull/26910

Please boost for reach! I think this is an important feature that would help people understand Mastodon better.

I even wrote a detailed bug AND design doc, that references other similar issues: https://github.com/mastodon/mastodon/issues/26863

Completely ghosted by the Mastodon team so far...😭

#mastodon #meta #github #CodeReview #foss #OpenSource

harrysintonen, to machinelearning

8.5.0 will ship a fix for a I discovered – this is the 26th vulnerability I’ve found in the project. Yet, I must again emphasize my firm belief that curl is an extremely robust project: none of the issues I’ve found have been high or critical severity ones. Most of the issues have been in the application logic, and logic flaws is one of the last class of vulnerabilities that automation struggles to find. Even struggles, if the impact of the vulnerability isn’t an out of bound memory access (and thus crash), or if the interaction is a complex one. There is no easy way out with logical flaws: Vigilance and manual are the only remedies that really work. Someday and similar technologies are likely to reach a level that they could perform some of these task – however for now this is one of the very few areas that automation haven't been able to touch.

  • All
  • Subscribed
  • Moderated
  • Favorites
  • megavids
  • thenastyranch
  • rosin
  • GTA5RPClips
  • osvaldo12
  • love
  • Youngstown
  • slotface
  • khanakhh
  • everett
  • kavyap
  • mdbf
  • DreamBathrooms
  • ngwrru68w68
  • provamag3
  • magazineikmin
  • InstantRegret
  • normalnudes
  • tacticalgear
  • cubers
  • ethstaker
  • modclub
  • cisconetworking
  • Durango
  • anitta
  • Leos
  • tester
  • JUstTest
  • All magazines