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

Run specs in CI #250

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

Run specs in CI #250

wants to merge 8 commits into from

Conversation

headius
Copy link
Member

@headius headius commented May 23, 2024

These are not all green but it's better to have them running than not.

headius added 7 commits May 23, 2024 09:01
import as a top-level function was removed years ago.
Probably intended to leave these as nil, but they don't handle nil
Not needed here anyway
These were not real failures; rather they are changes in how rspec
or JRuby works since the specs were last green.
@headius headius added this to the 1.2.0 milestone May 23, 2024
@headius headius force-pushed the run_tests_in_ci branch from 08b03c1 to d110340 Compare May 23, 2024 16:13
@headius headius modified the milestones: 1.2.0, 1.2.3 Jun 10, 2024
@chadlwilson
Copy link
Contributor

@headius I think superseded by #262 (although I did not cherry pick from here I believe as don't think I noticed this PR so you might want to check if there is anything here that is a better approach than mine?)

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.

2 participants