Closed
Conversation
dependency updates Pagy uses pagy.limit now for number of items on page updating depencies, moving to rails 8.0.4, ruby 3.4 and pagy 43 increased memory for tests increased memomory for app for tests, only run tests if setup succeeds changed based url for IIF server, removed memory allocation for compose.ci using selenium stand alone chrome, broke out steps in build.yml for precompile and db:prepare bringin up selenium after assets are precompiled, lowering the reserved memory for selenium bypassing yarn build -watch in ci since assets are precompiled in a different step only start selenium before rspec needs it bringing up compose without dependencies so sellenium doesn't start up early running precompile as a standalone. hope this fixes memory issue when building during test using dummy key_base for precompile for rspec need to have iipsrv, app and selenium running for rspec need to have iipsrv, app and selenium running need bundle exec when calling rspec and rubocop full rails server not needed for tests, using slimmed down version for tests running system tests as stand alone adding sleep infinity to docker-compose.ci, hopefully fixes memory issue running tests refactored do_profile in application_controller Added omniauth-rails_csrf_protection gem and updated omniauth intializer for post only removed commented out code, set config.yjit false and removed sleep infity from compose.ci ran rails app:update and made necessary changes to accommodate the update to rails 8 added sleep infinity back in to compose.ci, tests failing in build without it changed unauthorized user redirect so it goes to a page instead of directly to calnet removed some Gem dependencies, changed a comment in puma config, added static page instead of direct redirect to CAS fixed a fex minor styling issues in specs removed autoload of lib/assets as we dont' use that directory set rubocop and rspec to always run, removed commented sections out of docker-compose, remove unused logger setting for production, removed redundant to_s in borrower_toker.rb reverting build.yml to 1.7.9 release version, removed sleep infinity from compose.ci Added dummy secret for build.yml starting rails server in compose.ci so it doesn't run yarn build removed an outdated comments, some formatting changes in specs, put back in the build steps to handle memory issues added newline to end of spec fixed rubocopy formatting error using 3.3 instead of 3.4, changes for chrome driver settings, spec Added --add-platform x86_64-linux to Gemfile.lock Accounting for secret key base in build having docker image generate rails secret for SECRET_KEY_BASE Update .github/workflows/build.yml Co-authored-by: Anna Wilcox <AWilcox@Wilcox-Tech.com> Update docker-compose.ci.yml Co-authored-by: Anna Wilcox <AWilcox@Wilcox-Tech.com> Update docker-compose.ci.yml Co-authored-by: Anna Wilcox <AWilcox@Wilcox-Tech.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated CHANGES.md