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.

Sign up for GitHub

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

Jump to bottom

Update to rails 7 #377

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Update to rails 7 #377

wants to merge 40 commits into from

Conversation

sitarane
Copy link

@sitarane sitarane commented Aug 3, 2023

  • Update both ruby and rails to current versions
  • Add named volumes to compose file
  • Add an .env file
  • Add a separate container for the test environment
  • Add an email client container for apps that send email (plus config steps)
  • Add a web browser container for system tests (plus config steps)
  • Replace the old rails5 screenshot with the current one

@sitarane
Update README.md
9d8bd0e 
Update Dockerfile

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
4a57b87 
Comment out the precompile-asset line in Dockerfile. Only used in prod.

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
f827295 
Change rails version to 7

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
f94f3b3 
Rename myapp > app in entryfile

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
0ccebd6 
Update docker-compose

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
b03eacb 
Add .env file

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
0f3fe7e 
Add some structure to the document

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
48013dd 
Set up the database connection in rails.

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
ace6271 
Create the database

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
update screenshot of welcome screen
eb237f3 
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
789e921 
Comment out the compose file section about the debugger.

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
091dea6 
Add some useful commands

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
89520d2 
Edit "up" command so it doesn't start unnecessary services.

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Remove unecessary packages from the container.
95af8a3 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
add configure mail section
89b9ebb 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Add section about tests WIP
d27d19d 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Full setup for system tests
245c77a 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Add the service to compose up commands where it makes sense
221979c 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Add some useful commands
8e2397a 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Update README.md
667b76f 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Add versions to external containers to avoid breaking changes
f63bc8f 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Unpin ruby version
a0c5350 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Set ruby version to current
b296cde 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Improve the comments in the db setup
abed0c6 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Be more specific in a compose up command
0a09375 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Specify maildev version
b49009a 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Specify chrome version
69f123e 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Add step to migrate test database
48e28fa 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Set up a scaffold
d246776 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Preface the Tests section
ff549ad 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Fix command typo
336f6c6 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Simplify the test section by switching to headless_chrome
1a197e4 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Remove the open port on the chrome server
1cfeb65 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Remove unnecessary gem from gemfile
f19b83e 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Make app minimal by default
5d3f6c8 
User needs to uncomment the mail client and the test browser if they need it.

Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Comment out the Depend statement for commented out services
05bff78 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Reformulate driver definition section
48d55ec 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
remove unnecessary build
d44427b 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
Remove console output lines
d19ff69 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane
fix a wrong language tag
beec100 
Signed-off-by: Julien Boyer <julien@larevolution.re>
Signed-off-by: sitarane <julien@larevolution.re>
@sitarane sitarane mentioned this pull request Aug 3, 2023
Open
@TelmoNeto TelmoNeto mentioned this pull request Dec 19, 2023
Open
@Gemy-star Gemy-star mentioned this pull request Dec 19, 2023
Open
@Nikunz Nikunz mentioned this pull request Dec 19, 2023
Open
@marcosRandulfe marcosRandulfe mentioned this pull request Dec 19, 2023
Open
This was referenced Dec 20, 2023
Open
Open
@ntnguyencse ntnguyencse mentioned this pull request May 18, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stewartmbofana stewartmbofana stewartmbofana approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sitarane @stewartmbofana