Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(CAT-2094) Reusable shellcheck #272

Merged
merged 3 commits into from
Oct 15, 2024
Merged

Conversation

LukasAud
Copy link
Contributor

@LukasAud LukasAud commented Oct 9, 2024

Shellcheck has been moved to our reusable workflows repository and is
now part os our CI testing. As such, we are removing the action from
here and enabling it to be called.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@LukasAud LukasAud requested a review from a team as a code owner October 9, 2024 07:55
@LukasAud LukasAud marked this pull request as draft October 9, 2024 07:58
@LukasAud
Copy link
Contributor Author

LukasAud commented Oct 9, 2024

Shellcheck has been moved into gem_ci, not module_ci. Therefore, this change would not work as of now. A decision will be made soon on whether to implement this check for modules as well or not. Until then, do not merge this.

@LukasAud LukasAud changed the title (CAT-2013) Reusable shellcheck (CAT-2094) Reusable shellcheck Oct 14, 2024
Shellcheck has been moved to our reusable workflows repository and is
now part os our CI testing. As such, we are removing the action from
here and enabling it to be called.
@LukasAud LukasAud force-pushed the CAT-2013-reusable_shellcheck branch from f91a01c to bff80cf Compare October 15, 2024 12:33
@LukasAud
Copy link
Contributor Author

Shellcheck has now been moved into module_ci.yml. Commit 71acc3f was used for testing, now undone. This PR is ready to merge.

@LukasAud LukasAud marked this pull request as ready for review October 15, 2024 12:38
@jordanbreen28 jordanbreen28 merged commit 4397f5c into main Oct 15, 2024
5 checks passed
@jordanbreen28 jordanbreen28 deleted the CAT-2013-reusable_shellcheck branch October 15, 2024 13:00
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.

2 participants