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

Prepare for refactoring tests #124

Merged
merged 9 commits into from
Jun 29, 2024
Merged

Conversation

jrmhaig
Copy link
Collaborator

@jrmhaig jrmhaig commented Mar 28, 2024

  1. Notes on plan for refactoring tests
  2. Create pipeline to run Rubocop

jrmhaig and others added 5 commits March 28, 2024 15:22
All supported versions of Active Record can now use sqlite3 version 1.6 so
there is no need to explicitly set it in each gemfile.
The 'default' option was changed to defined a long time ago.
Minor updates of Rubocop and RSpec can cause unexpected failures in the
pipeline that are not related to the changes that are being made. To avoid this
updates are restricted to patch updates.
@jrmhaig jrmhaig force-pushed the test_refactor_notes branch 4 times, most recently from d9511ab to 62136a9 Compare March 29, 2024 17:57
jrmhaig added 2 commits April 1, 2024 19:34
Tests for the unreleased version of Active Record are now in a separate file
so that a failure does not cancell other tests. Really, Github Actions should
have an 'allow failure' option.
@jrmhaig jrmhaig force-pushed the test_refactor_notes branch from 4b175d3 to f32c545 Compare April 1, 2024 18:34
@jrmhaig jrmhaig force-pushed the test_refactor_notes branch 3 times, most recently from d645a1e to 9a13801 Compare June 21, 2024 19:43
@jrmhaig jrmhaig force-pushed the test_refactor_notes branch from 9a13801 to 8a91618 Compare June 21, 2024 19:46
@jrmhaig jrmhaig force-pushed the test_refactor_notes branch from 4b75bc6 to d05c298 Compare June 21, 2024 20:30
@jrmhaig jrmhaig merged commit ca0ab62 into amoeba-rb:master Jun 29, 2024
23 checks passed
@jrmhaig jrmhaig deleted the test_refactor_notes branch June 29, 2024 15:18
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.

1 participant