Skip to content

Add Rubycritic Gem #2208

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

Closed
wants to merge 2 commits into from
Closed

Add Rubycritic Gem #2208

wants to merge 2 commits into from

Conversation

tiagomenegaz
Copy link
Collaborator

@tiagomenegaz tiagomenegaz commented Feb 11, 2025

What are you trying to accomplish?

This PR aims to add rubycritic with vanilla configurations since Add rubycritic gem is one of the items listed here: #2085

image

Run the following command

bundle install
rubycritic

What approach did you choose and why?

Basic install using the existing gemspec without any additional configuration.

Anything you want to highlight for special attention from reviewers?

It would be great to integrate this change with the existing pipelines. However, it could be done in a different PR.

@tiagomenegaz tiagomenegaz self-assigned this Feb 11, 2025
@Spone
Copy link
Collaborator

Spone commented Feb 12, 2025

Hi @tiagomenegaz thanks for contributing! Can you elaborate on the rationale behind this change? I'm not sure why you suggest adding this dependency.

@tiagomenegaz
Copy link
Collaborator Author

tiagomenegaz commented Feb 12, 2025

Hi @tiagomenegaz thanks for contributing! Can you elaborate on the rationale behind this change? I'm not sure why you suggest adding this dependency.

Hi @Spone . Thanks for your comment. I'm adding this dependency because this is one of the items listed here: #2085 by @joelhawksley . I'll add this to this PR's description for future reference and clarification.

@tiagomenegaz tiagomenegaz changed the base branch from remove-respond-to to v4 March 3, 2025 22:29
@joelhawksley joelhawksley marked this pull request as draft March 10, 2025 18:43
@joelhawksley
Copy link
Member

joelhawksley commented Mar 10, 2025

@tiagomenegaz thanks for opening this PR! I've looked through the output from running rubycritic and I think we should consider adding each of its pieces separately, as it runs simplecov in a manner that appears to be incompatible with how we're using it already.

I hope to be able to revisit this soon and figure out a path forward ❤️

@tiagomenegaz
Copy link
Collaborator Author

I hope to be able to revisit this soon and figure out a path forward ❤️

Sure. I'll close this PR to keep everything clean. 🤞

@tiagomenegaz tiagomenegaz deleted the add-rubycritic-gem branch March 11, 2025 21:17
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.

3 participants