)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0a1317f2565e92883a064d684f0ee4dc508828e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"51e8d700_50921d95","updated":"2022-02-04 11:05:32.000000000","message":"I\u0027d rather change tox.ini to not enforce this than undo that part of the change. The only other *rc file that we still have left is \"openrc\", guess we can match that explicitly. I\u0027m going to do a new version unless Sean is faster ...","commit_id":"53044ca74c60e366f8e4b403388f6ec6ade1ca43"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7b2e45c58a044a10f481e096193b6b66c34b590b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"04acc5d5_878fdc5d","updated":"2022-02-04 08:43:36.000000000","message":"Thx, I\u0027ve been meaning to do this myself for quite some time.","commit_id":"53044ca74c60e366f8e4b403388f6ec6ade1ca43"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"3b26d199542948d4bfcf63747c54ad01ac6310ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b9d177f6_3bfb6bef","updated":"2022-02-04 10:54:21.000000000","message":"bashate fails","commit_id":"53044ca74c60e366f8e4b403388f6ec6ade1ca43"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"5aae197132ad54c1b3b4fa1a8c6df5298115c054","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6b9b32a2_4fdb92d2","updated":"2022-02-04 12:06:04.000000000","message":"Actually removing the shebang line is a more complex topic, so I\u0027ve dropped it from this patch for now.\n\nI still think that we would want to remove it applying the reasoning that Sean gave, but that same reasoning would seem to apply to everything in inc/**, lib/** and possibly others, so that would be a larger cleanup, which should be discussed first. I\u0027ll put it onto the agenda for the next QA meeting.","commit_id":"85c7d8db4eef2e367a7466a39b4f1fba7a983eef"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"918fce57d1d9686195e07e08df92e9e7fc16201b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2462d275_a31ab3f7","in_reply_to":"6b9b32a2_4fdb92d2","updated":"2022-02-04 20:16:45.000000000","message":"ack ya, i just got in the habit of removing it since in other repos I use pre-commit to enforce that if it has a shebang line it should be executable and if it is executable it should have a shebang line.\n\nso it was partly habit and partly just to not imply you should actually run it since it should generally be sourced if you are going to use it.","commit_id":"85c7d8db4eef2e367a7466a39b4f1fba7a983eef"}]}