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

Change dependency from annotate to annotaterb #30232

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

drwl
Copy link

@drwl drwl commented May 10, 2024

This PR changes the project to use AnnotateRb instead of Annotate.

I tested it locally and there are no changes to the annotation configuration nor annotations themselves. What does change is the CLI, so if running annotations manually previously you would use bundle exec annotate, now one would use bundle exec annotaterb models.

I was a maintainer of the old gem (annotate) but decided last year to make a fork and tidy up the code a lot. The new gem (annotaterb) is actively maintained and should perform faster for newer Rails (6+) apps because it leverages Zeitwerk instead of eager loading everything.

This PR is self-serving in that it does grow usage of my project, so feel free to reject this. I am also happy to answer any questions.

@renchap renchap added dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code labels May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants