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

Switch to SQLite3, Upgrade Ruby and Rails, Remove Unused Gems, and Add Fly.io Deployment Configuration #89

Merged
merged 8 commits into from
Jul 30, 2024

Conversation

cirdes
Copy link
Contributor

@cirdes cirdes commented Jul 27, 2024

New server: https://rbui.fly.dev

This PR includes the following updates and changes:

  1. Database Switch:

    • Removed PostgreSQL and Redis, and switched to SQLite3 for simpler web stack.
  2. Version Upgrades:

    • Updated Ruby to version 3.3.4.
    • Updated Rails to version 7.2.0.beta3.
  3. Asset Pipeline Update:

    • Removed Sprockets and added Propshaft for managing assets.
  4. Removed Unused Gems:

    • Removed the following unused gems:
      • stripe
      • letter_opener (development group)
      • postmark-rails (used in production)
      • octokit (version ~> 5.0)
      • faraday-retry (version ~> 2.0) - used for API calls failing in the octokit gem
      • rack-www - used to redirect www to non-www subdomain
      • rollbar
  5. Deployment Configuration:

    • Added configuration for deployment on Fly.io.

These changes streamline the project dependencies, simplify development, and prepare the application for deployment on Fly.io.

@cirdes cirdes changed the title Adding sqlite3 Switch to SQLite3, Upgrade Ruby and Rails, Remove Unused Gems, and Add Fly.io Deployment Configuration Jul 27, 2024
Copy link
Collaborator

@SethHorsley SethHorsley left a comment

Choose a reason for hiding this comment

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

almost perfect

.yarn/releases/yarn-1.22.19.cjs Outdated Show resolved Hide resolved
.yarnrc Outdated Show resolved Hide resolved
Gemfile Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
@cirdes cirdes requested a review from SethHorsley July 29, 2024 15:03
@cirdes
Copy link
Contributor Author

cirdes commented Jul 29, 2024

@iseth, I have removed the ".tool-versions" file and fixed the comment in the Gemfile.

All other issues you mentioned were generated by the fly.io CLI, and I believe they are recommended by Yarn.

Let me know if you need anything else!

@SethHorsley
Copy link
Collaborator

@iseth, I have removed the ".tool-versions" file and fixed the comment in the Gemfile.

All other issues you mentioned were generated by the fly.io CLI, and I believe they are recommended by Yarn.

Let me know if you need anything else!

I dig too deep into this but i think its definetly coming from https://github.com/fly-apps/dockerfile-rails/blob/2711cf5aab3716c3dafd1c08307ee62107a45dda/lib/generators/templates/_npm_install.erb

@cirdes
Copy link
Contributor Author

cirdes commented Jul 30, 2024

@iseth, I have removed .yarn/* and .yarnrc and added them to .gitignore.

Copy link
Collaborator

@SethHorsley SethHorsley left a comment

Choose a reason for hiding this comment

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

awesome!

@SethHorsley SethHorsley merged commit dc14a12 into main Jul 30, 2024
4 checks passed
@SethHorsley SethHorsley deleted the cirdes/move-to-sqlite-and-fly branch July 30, 2024 21:19
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