Skip to content

Conversation

ali-kafel
Copy link
Contributor

Summary

The action defined in this repo is not used anywhere. We are mainly only referencing the checkstyle configurations in downstream repositories. We are removing all references to the unused action.

Detail

Removal of GitHub Action and related files:

  • Deleted the Dockerfile, which previously set up the environment for running Checkstyle and Reviewdog.
  • Removed the GitHub Action definition from action.yml, including all inputs and Docker configuration.
  • Deleted the entrypoint.sh script that executed Checkstyle and piped results to Reviewdog.

Documentation update:

  • Updated README.md to reflect the new purpose of the repository, focusing on providing Checkstyle configurations rather than an action.

Testing

No testing needed we are simply only referencing the already existing checkstyle configs. I checked all repos and we are not using the action anywhere.

Documentation


Requested Reviewers: @mention

Copy link
Contributor

@creilly12 creilly12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

Confirmed not in use. This was a review bot that didn't pan out.

@creilly12 creilly12 merged commit fee5d29 into circlefin:master Sep 12, 2025
3 checks passed
@ali-kafel ali-kafel deleted the remove-action branch September 12, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants