Skip to main content
132 votes
Accepted

Would it be a bad idea to periodically run code formatters on a repository?

Sounds nice, but I would prefer to have people responsible for committing code changes, not bots. Besides, you want to make absolutely sure that those changes do not break anything. For example, we ...
Marcel's user avatar
  • 3,112
75 votes

Would it be a bad idea to periodically run code formatters on a repository?

I would instead try to make it really easy for everyone in the team to apply automatic code formatting according to your team's standard directly inside your editor or IDE, to the current source code ...
Doc Brown's user avatar
  • 208k
37 votes

Would it be a bad idea to periodically run code formatters on a repository?

It's a bad idea, not just because it discourages people from writing decent code, but also because the reformatting will show up as code changes in your VCS (you do use one I hope), masking the ...
jwenting's user avatar
  • 9,783
29 votes

Would it be a bad idea to periodically run code formatters on a repository?

I would tend to believe it is a good idea (to automatically run code formatters), but that is just my opinion. I won't run them periodically, but if possible before version control commits. With ...
Basile Starynkevitch's user avatar
17 votes

Would it be a bad idea to periodically run code formatters on a repository?

In general, I think it is a bad idea. In principle it's a valid idea, but it can be problematic in reality. Having the code formatter break your code is a real possibility, and it only takes one ...
Karl Nicoll's user avatar
17 votes

Would it be a bad idea to periodically run code formatters on a repository?

I think it's a bad idea. Many of the answers already covered that it dirties the history by making it difficult to pin down who actually added a line and that it encourages people to just commit ...
Captain Man's user avatar
10 votes

Would it be a bad idea to periodically run code formatters on a repository?

Yes, I think it's a bad idea. Don't get me wrong, the reason to do it sounds great, but the result could still be horrible. You will have merge conflicts when pulling a tracked branch, atleast I fear ...
HopefullyHelpful's user avatar
7 votes

Would it be a bad idea to periodically run code formatters on a repository?

Something I haven't seen mentioned is the fact that sometimes there are legitimate reasons to not format something according to a set of rules. Sometimes the clarity of the code is improved by going ...
Andy Lester's user avatar
  • 4,810
7 votes

Would it be a bad idea to periodically run code formatters on a repository?

It’s an awful idea. If one of my developer colleagues made gratuitous changes to source files, it wouldn’t pass a code review. It just makes life harder for everyone. Change the code that needs ...
gnasher729's user avatar
  • 45.4k
7 votes

Why only scripting languages for automating tasks?

Scripting is automation. Scripting languages were specifically developed for automation tasks. They often have features that make scripting easy, e.g. simple mechanisms for invoking other programs or ...
amon's user avatar
  • 134k
7 votes

Does linting without auto-correction make sense?

What would be the alternative? No linter at all, and thus no style fixes at all - except for the ones found during code reviews. But obviously - the latter have to be applied manually as well. So ...
Doc Brown's user avatar
  • 208k
7 votes

The "real and effective" GIT CI/CD strategy

By having three repo you kind of have a branching strategy already but a complex one. I think you're not facing a technical issue but a cultural issue. Even if your solution is good (I think it is but ...
JayZ's user avatar
  • 827
6 votes

How do you adapt to changes to product specifications during development?

Your parser approach is definitely a step in the right direction. To deal with what you describe, you need to follow two well known software engineering principles as rigidly as possible: 1. Single ...
Doc Brown's user avatar
  • 208k
6 votes

How to convert from TDD scripts to BDD?

In practice, you can´t, because the concepts of TDD and BDD are different. Although both test behavior, when you use TDD you create low-level testing to ensure your methods/function is working as ...
Edu Costa's user avatar
  • 330
6 votes

What is needed to safely enable non-technical users to trigger automatic deployments?

It is actually more than just triggering the pipeline job to start. If the job fails how should we know if it's the bug from your artifact, resource shortage, or some transient error from the ...
lennon310's user avatar
  • 3,212
5 votes

SBCL development on Windows, including CFFI

This is a little late, but as of about 2 months ago (05/2018) I found this project called Portacle which bundles SBCL, Emacs, Quicklisp, and more into a single package for Windows 10. All you have to ...
λ Jonas Gorauskas's user avatar
5 votes

What advantages do daily builds provide specifically over continuous builds in practice?

I work on a very large project with dozens of software engineers, dozens of QA engineers, etc. Our codebase is several million lines of code. We have nightly builds and we have continuous builds. They ...
user1118321's user avatar
  • 4,993
5 votes
Accepted

How do I automate build-publish tasks for my web server?

A Bash script is exactly how it happens in many companies. Avoid steps that you have to perform manually. Automate them so that you only have to kick off the process and can then sit back. Being able ...
amon's user avatar
  • 134k
4 votes
Accepted

Where to run automation UI Tests - Build server or Test machine

Typically a build machine is meant for build, compile + unit tests. There should be no external dependencies. Usually, UI automation happens as a post deployment step after the package has been ...
Jon Raynor's user avatar
  • 11.5k
4 votes

Would it be a bad idea to periodically run code formatters on a repository?

You don't mention the VCS you use but depending on that another option is to have a server-side hook. A VCS like git supports that. You could install a sever-side hook that runs the formatter on the ...
sigy's user avatar
  • 716
4 votes

Best Practices for Automating Word Documents

Gain control of how report data is formatted. Excel simply isn't a data format. It has it's own presentation ideas. You can make this work and it sounds like a lot has already been built help you ...
candied_orange's user avatar
4 votes

What exactly caused the recent increase in automated testing?

It is opinionated question of why there is such percieved increase of automated testing. Especially automated testing done by developers, not testers. Automated testing is as old as software ...
Euphoric's user avatar
  • 37.4k
3 votes

Authenticate Git Pulls on a Compromised Machine

In general, once a machine is compromised the layer at which it has been compromised must be replaced to achieve any guarantees of authenticity from that layer. Say for example a user's home directory ...
l0b0's user avatar
  • 11.4k
3 votes
Accepted

Problem in creating a correct finite-state-machine for a given short text

Some random thoughts: You're not representing the individual stories in the state machine, thus, either you'll have to track them "externally" to the bubbles/states of the state machine, or, ...
Erik Eidt's user avatar
  • 33.8k
3 votes
Accepted

PowerShell performance when running Excel macros

Powershell is based on the .NET framework infrastructure and runtime, which is pretty efficient and to my own experience often more effective in memory management and CPU utilization than the old COM-...
Doc Brown's user avatar
  • 208k
3 votes

Partially Automated Migration of Monolith to MicroService

It is very surprising to me that I can not find even one tool that analyses the way a monolith accesses the data base and suggests something about how cutting of certain areas may help in migrating ...
CodeScale's user avatar
  • 196
3 votes

How to identify scope of end-to-end automation testing?

Your tests is incomplete. It shouldn't stop there. Step 4. Find the entity created by step 1-3. Made easier if Step 3 also return ID of a created entity. Step 5. Open details of the entity. Step 6....
Euphoric's user avatar
  • 37.4k
3 votes

What is needed to safely enable non-technical users to trigger automatic deployments?

Your question contains parts of the answer. The deployment by that non-technical person must be monitored and rolled back automatically. Basically, it means automation of the workload you do while ...
Toshakins's user avatar
3 votes

What is needed to safely enable non-technical users to trigger automatic deployments?

You need some sort of approval system in place so technical people and business people can understand what will be deployed, when, and what impact it has on the wider technology ecosystem. Your friend ...
Greg Burghardt's user avatar
3 votes
Accepted

To lint on the client, or the server?

Because people (definitly includes me) have the tendency to focus on "delivering the feature", they tend to sometimes forget the quality assurance steps. Therefor i add the linting always to ...
JanRecker's user avatar
  • 1,576

Only top scored, non community-wiki answers of a minimum length are eligible