rabajaj0509 commented on code in PR #31: URL: https://github.com/apache/skywalking-terraform/pull/31#discussion_r1376455484
########## ansible/roles/install-java/tasks/main.yml: ########## @@ -15,19 +15,19 @@ --- - name: Install Java 11 on RHEL-based systems - package: + ansible.builtin.package: Review Comment: From what I understand, if the linter suggests we must be explicit then let's be explicit. Also, I think the linter would work on PRs as we make changes to files. So let us not touch all the files and make changes rather with time as we make changes to playbooks, the linter will pop up and state modules explicitly. What do you think? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
