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

Merging from release 1.1.1 published on gitlab.com/no_reply/ActiveTriples #8

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

jrgriffiniii
Copy link

This is necessary to advance the maintenance of a fork used by the Samvera Community for resolving samvera/active_fedora#1470 (comment) with a new release.

Tom Johnson and others added 13 commits January 4, 2019 09:34
We support currently released Ruby versions from 2.1. The CI pipeline should
test each of these. We decline to test 2.1, due to the complexity of the
setup. In principle, it should still work, though we should probably drop
support soon.

JRuby 9.5 is also supported.
Test ruby versions 2.1+

See merge request no_reply/ActiveTriples!281
GitLab supports parsing coverage info directly from build output. This switches
to using that mechanism instead of the network-dependant coveralls.
Use `SimpleCov` directly for coverage status

See merge request no_reply/ActiveTriples!280
Pipeline status

See merge request no_reply/ActiveTriples!282
the old implementation gsubed the entire method list over and over again. it's
unspeakably, horrifyingly faster to mutate the name we're going to check once.

nothing scientific about this, but it speeds up my local test suite runs by at
least a third.
faster protected property detection

See merge request no_reply/ActiveTriples!283
prepare release 1.1.1

See merge request no_reply/ActiveTriples!284
@jrgriffiniii jrgriffiniii marked this pull request as draft June 29, 2022 16:09
@cjcolvar cjcolvar mentioned this pull request Jan 11, 2023
18 tasks
@jrgriffiniii jrgriffiniii marked this pull request as ready for review January 13, 2023 14:42
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