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

Upgrade backend #28

Merged
merged 24 commits into from
Nov 16, 2023
Merged

Upgrade backend #28

merged 24 commits into from
Nov 16, 2023

Conversation

JuzerShakir
Copy link
Owner

@JuzerShakir JuzerShakir commented Nov 15, 2023

This PR includes the following changes:

Gems

Gem Before Now
bundler 2.3.26 2.4.21
bcrypt 3.1.18 3.1.19
capybara 3.38.0 3.39.2
faker 3.1.0 3.2.2
friendly_id 5.5.0 5.5.1
pagy 6.0 6.2
pg 1.4.5 1.5.4
puma 5.6.5 6.4.0
rails 7.0.4.3 7.1.2
ransack 3.2.1 4.1.1
redis 4.0 5.0.8
ruby 3.1.2 3.2.2
selenium-webdriver 4.7 4.15
simple_form 5.1 5.3

Tests

Updating the capybara & selenium-webdriver gems have significantly decreased the runtime of tests.

Metric Before Now Percentage Improvement
Load time ~1.6 secs ~1.2secs 25%
Run time ~48 sec ~40 sec 16.67%

CI

We can see similar performance improvement in the runtime of GitHub Action (lint-n-test):

PR Before After Percentage Improvement
Rails Way ~4min 40 sec
Update Fly.io & CI ~2min 40 sec 42.9%
This PR ~2min 40 sec ~1min 57secs 26.9%

Overall improvement is 58.21% from ~4mins 40secs to ~1min 57secs.

@JuzerShakir JuzerShakir merged commit 1d7f15b into master Nov 16, 2023
1 check passed
@JuzerShakir JuzerShakir deleted the upgrade-backend branch January 30, 2024 05:51
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.

1 participant