Skip to content

Forbid using standard streams in archunit#11130

Merged
wing328 merged 1 commit intoOpenAPITools:masterfrom
agilob:archunit-forbid-using-standard-streams
Dec 17, 2021
Merged

Forbid using standard streams in archunit#11130
wing328 merged 1 commit intoOpenAPITools:masterfrom
agilob:archunit-forbid-using-standard-streams

Conversation

@agilob
Copy link
Copy Markdown
Contributor

@agilob agilob commented Dec 15, 2021

new archunit test check for not allowing to use standard streams like System.out and System.err, this PR replaces #11128

@wing328 @jimschubert

@kuhnroyal
Copy link
Copy Markdown
Contributor

Interesting, I am out of the Java world for too long.

@agilob
Copy link
Copy Markdown
Contributor Author

agilob commented Dec 15, 2021

This test will scan generated byte code and check if you're breaking defined rules. You can prevent a lot nasty things with archunit, make sure you're not using wrong classes, deprecated code, code known to be slow. With a few unit tests you can prevent people from writing things that are known to be "bad" or controversial.

@wing328 wing328 added this to the 5.3.1 milestone Dec 17, 2021
@wing328 wing328 merged commit ebb6914 into OpenAPITools:master Dec 17, 2021
@agilob agilob deleted the archunit-forbid-using-standard-streams branch December 17, 2021 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants