-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature#01 initial setup #3
Conversation
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
…ments Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]> Co-authored-by: JorgeDDW <[email protected]>
Co-authored-by: marianoalarcon <[email protected]> Co-authored-by: facundotoconas <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are ready to begin
Gemfile
Outdated
# Call 'byebug' anywhere in the code to stop execution and get a debugger console | ||
gem 'byebug', platforms: %i[mri mingw x64_mingw] | ||
gem "rspec-rails", "~> 5.0" | ||
gem "sqlite3", "~> 1.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps can you remove this gem , I noticed you are not using it in config/database.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right, thanks! in principle we thought to use sqlite3 for development and testing, in this way we keep postgres only for production, but then we choose to keep everything in postgres
gem 'rake', '~> 13.0' | ||
gem 'rubocop-rails', '~> 2.10' | ||
gem 'rubycritic', '~> 4.6' | ||
gem 'rubycritic-small-badge', '~> 0.2.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could it be that you forgot to configure this gem? or are you going to do it later in other issue?
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
…pec_helper Co-authored-by: SpidySamurai <[email protected]> Co-authored-by: facundotoconas <[email protected]>
Gemfile
Outdated
@@ -52,9 +52,13 @@ group :development do | |||
gem 'rubocop-rails', '~> 2.10' | |||
gem 'rubycritic', '~> 4.6' | |||
gem 'rubycritic-small-badge', '~> 0.2.1' | |||
gem "hamlit", "~> 2.15" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes ! This gem improves the dev !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I see you included several things, it seems that my previous review was gone
--format console | ||
) | ||
task.paths = FileList["#{dir}/**/*.rb"] | ||
RubyCritic::RakeTask.new("_rubycritic:#{dir}") do |task| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice , I am glad that you liked this approach 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we liked the order you keep when making the project settings. We find it efficient and effective
]) | ||
end | ||
SimpleCov.minimum_coverage(65) | ||
SimpleCovSmallBadge::Formatter.class_eval do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This monkey patch is very useful without this gem fails , can see context here: MarcGrimme/simplecov-small-badge#15
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohh, thank u very much, knowing these kind of stuff helps a lot!
Description
Non-functional requirements
Configures rubocop , rubycritic, sandi_meter as code review tools for quality
Setup tests
Configures rspec-rails for testing.
Setup frontend
Configures bootstrap 5 as frontend framework.
Preconditions
Execute metrics code and sandi_meter
rake metrics_code