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

CI fix #434

Merged
merged 12 commits into from
Apr 11, 2024
Merged

CI fix #434

merged 12 commits into from
Apr 11, 2024

Conversation

ublefo
Copy link
Contributor

@ublefo ublefo commented Mar 27, 2024

Various CI related fixes and improvements:

  1. Implemented container caching to bring turnaround time from 52 minutes to 16 minutes after the container is built successfully.
  2. Set up redis in the workflow and make sure sidekiq is configured to use the correct URL from the environment if it's present.
  3. Upgrade all workflows to the latest versions.
  4. Install ruby-lsp in the development environment for the corresponding vscode extension, which is currently already installed in the devcontainer but completely useless due to the gem being missing.
  5. Implement better error reporting in database_populator.rb to report the error messages when there's something wrong with the test data. Troubleshooting test db initialization issues should be much easier now.
  6. Documented the two new redis related env variables.
  7. Skip unit tests for documentation updates.
  8. Created a dependabot config to check for github actions workflow updates weekly.

@maddernd
Copy link
Contributor

Approved by DM, just waiting on @macite approval.

@macite macite merged commit 18136c2 into doubtfire-lms:development Apr 11, 2024
4 checks passed
@ublefo ublefo deleted the ci-fix branch April 15, 2024 03:29
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.

3 participants