noid-rails 3.0.3 → 3.2.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.circleci/config.yml +66 -38
- data/CONTRIBUTING.md +21 -12
- data/README.md +21 -5
- data/lib/noid/rails/engine.rb +12 -0
- data/lib/noid/rails/version.rb +1 -1
- data/noid-rails.gemspec +4 -3
- metadata +11 -10
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: ef47e4775b782124ad9b60851d5b1df1ce79ffb9941136a2816f3432f6b2ba7c
|
|
4
|
+
data.tar.gz: 1d2fa4172edf9a2eda5a5010da3909d981172866a7e22068f27799be6ec9ece9
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 9f2444a05c4eae9039caef7920a24f6b4c38fa8e0af0bdda112b10167b05ecff8c8c46e0e02817f567a7e2e3a37d6e06c5fe3900172785229e2c0703c72c57ed
|
|
7
|
+
data.tar.gz: 85e59c9680c4f36b02657044656934f37cf18fd9928770de1980caaf64a6481fb267abb78db6bbf8c0ab5f6ea123ce8a6de20f00f616d60196a44548cf7d2865
|
data/.circleci/config.yml
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
version: 2.1
|
|
2
2
|
orbs:
|
|
3
|
-
samvera: samvera/circleci-orb@0
|
|
3
|
+
samvera: samvera/circleci-orb@1.0
|
|
4
4
|
jobs:
|
|
5
5
|
bundle_and_test:
|
|
6
6
|
parameters:
|
|
@@ -8,7 +8,7 @@ jobs:
|
|
|
8
8
|
type: string
|
|
9
9
|
bundler_version:
|
|
10
10
|
type: string
|
|
11
|
-
default:
|
|
11
|
+
default: 2.4.3
|
|
12
12
|
rails_version:
|
|
13
13
|
type: string
|
|
14
14
|
executor:
|
|
@@ -21,6 +21,19 @@ jobs:
|
|
|
21
21
|
steps:
|
|
22
22
|
- samvera/cached_checkout
|
|
23
23
|
|
|
24
|
+
- run:
|
|
25
|
+
name: Check for a branch named 'master'
|
|
26
|
+
command: |
|
|
27
|
+
git fetch --all --quiet --prune --prune-tags
|
|
28
|
+
if [[ -n "$(git branch --all --list master */master)" ]]; then
|
|
29
|
+
echo "A branch named 'master' was found. Please remove it."
|
|
30
|
+
echo "$(git branch --all --list master */master)"
|
|
31
|
+
fi
|
|
32
|
+
[[ -z "$(git branch --all --list master */master)" ]]
|
|
33
|
+
|
|
34
|
+
- run: 'sudo apt-get update'
|
|
35
|
+
- run: 'sudo apt-get install -y libsqlite3-dev'
|
|
36
|
+
|
|
24
37
|
- samvera/bundle_for_gem:
|
|
25
38
|
ruby_version: << parameters.ruby_version >>
|
|
26
39
|
bundler_version: << parameters.bundler_version >>
|
|
@@ -42,53 +55,68 @@ workflows:
|
|
|
42
55
|
ci:
|
|
43
56
|
jobs:
|
|
44
57
|
- bundle_and_test:
|
|
45
|
-
name:
|
|
46
|
-
ruby_version:
|
|
47
|
-
rails_version:
|
|
58
|
+
name: ruby3-3_rails7-2
|
|
59
|
+
ruby_version: 3.3.0
|
|
60
|
+
rails_version: 7.2.0
|
|
48
61
|
- bundle_and_test:
|
|
49
|
-
name:
|
|
50
|
-
ruby_version: 2.
|
|
51
|
-
rails_version:
|
|
62
|
+
name: ruby3-2_rails7-2
|
|
63
|
+
ruby_version: 3.2.3
|
|
64
|
+
rails_version: 7.2.0
|
|
65
|
+
|
|
52
66
|
- bundle_and_test:
|
|
53
|
-
name:
|
|
54
|
-
ruby_version:
|
|
55
|
-
rails_version:
|
|
67
|
+
name: ruby3-3_rails7-1
|
|
68
|
+
ruby_version: 3.3.0
|
|
69
|
+
rails_version: 7.1.3
|
|
70
|
+
- bundle_and_test:
|
|
71
|
+
name: ruby3-2_rails7-1
|
|
72
|
+
ruby_version: 3.2.3
|
|
73
|
+
rails_version: 7.1.3
|
|
56
74
|
|
|
57
75
|
- bundle_and_test:
|
|
58
|
-
name:
|
|
59
|
-
ruby_version:
|
|
60
|
-
rails_version:
|
|
76
|
+
name: ruby3-3_rails7-0
|
|
77
|
+
ruby_version: 3.3.0
|
|
78
|
+
rails_version: 7.0.8
|
|
61
79
|
- bundle_and_test:
|
|
62
|
-
name:
|
|
63
|
-
ruby_version: 2.
|
|
64
|
-
rails_version:
|
|
80
|
+
name: ruby3-2_rails7-0
|
|
81
|
+
ruby_version: 3.2.3
|
|
82
|
+
rails_version: 7.0.8
|
|
83
|
+
|
|
84
|
+
- bundle_and_test:
|
|
85
|
+
name: ruby3-1_rails7-1
|
|
86
|
+
ruby_version: 3.1.4
|
|
87
|
+
rails_version: 7.1.3
|
|
65
88
|
- bundle_and_test:
|
|
66
|
-
name:
|
|
67
|
-
ruby_version:
|
|
68
|
-
rails_version:
|
|
89
|
+
name: ruby3-1_rails7-0
|
|
90
|
+
ruby_version: 3.1.4
|
|
91
|
+
rails_version: 7.0.8
|
|
69
92
|
|
|
70
93
|
- bundle_and_test:
|
|
71
|
-
name:
|
|
72
|
-
ruby_version:
|
|
73
|
-
rails_version:
|
|
94
|
+
name: ruby3-0_rails7-1
|
|
95
|
+
ruby_version: 3.0.6
|
|
96
|
+
rails_version: 7.1.3
|
|
74
97
|
- bundle_and_test:
|
|
75
|
-
name:
|
|
76
|
-
ruby_version:
|
|
77
|
-
rails_version:
|
|
98
|
+
name: ruby3-0_rails7-0
|
|
99
|
+
ruby_version: 3.0.6
|
|
100
|
+
rails_version: 7.0.8
|
|
78
101
|
- bundle_and_test:
|
|
79
|
-
name:
|
|
80
|
-
ruby_version:
|
|
81
|
-
rails_version:
|
|
102
|
+
name: ruby3-0_rails6-1
|
|
103
|
+
ruby_version: 3.0.6
|
|
104
|
+
rails_version: 6.1.7.6
|
|
82
105
|
- bundle_and_test:
|
|
83
|
-
name:
|
|
84
|
-
ruby_version:
|
|
85
|
-
rails_version:
|
|
106
|
+
name: ruby3-0_rails6-0
|
|
107
|
+
ruby_version: 3.0.6
|
|
108
|
+
rails_version: 6.0.6.1
|
|
86
109
|
|
|
87
110
|
- bundle_and_test:
|
|
88
|
-
name: ruby2-
|
|
89
|
-
ruby_version: 2.
|
|
90
|
-
rails_version:
|
|
111
|
+
name: ruby2-7_rails6-1
|
|
112
|
+
ruby_version: 2.7.8
|
|
113
|
+
rails_version: 6.1.7.6
|
|
91
114
|
- bundle_and_test:
|
|
92
|
-
name: ruby2-
|
|
93
|
-
ruby_version: 2.
|
|
94
|
-
rails_version:
|
|
115
|
+
name: ruby2-7_rails6-0
|
|
116
|
+
ruby_version: 2.7.8
|
|
117
|
+
rails_version: 6.0.6.1
|
|
118
|
+
- bundle_and_test:
|
|
119
|
+
name: ruby2-7_rails5-2
|
|
120
|
+
ruby_version: 2.7.8
|
|
121
|
+
rails_version: 5.2.8.1
|
|
122
|
+
|
data/CONTRIBUTING.md
CHANGED
|
@@ -67,15 +67,22 @@ further details.
|
|
|
67
67
|
|
|
68
68
|
* Fork the repository on GitHub
|
|
69
69
|
* Create a topic branch from where you want to base your work.
|
|
70
|
-
* This is usually the
|
|
71
|
-
* To quickly create a topic branch based on
|
|
72
|
-
* Then checkout the new branch with `git checkout fix/
|
|
73
|
-
* Please avoid working directly on the `
|
|
70
|
+
* This is usually the `main` branch.
|
|
71
|
+
* To quickly create a topic branch based on `main`; `git branch fix/main/my_contribution main`
|
|
72
|
+
* Then checkout the new branch with `git checkout fix/main/my_contribution`.
|
|
73
|
+
* Please avoid working directly on the `main` branch.
|
|
74
|
+
* Please do not create a branch called `master`. (See note below.)
|
|
74
75
|
* You may find the [hub suite of commands](https://github.com/defunkt/hub) helpful
|
|
75
76
|
* Make sure you have added sufficient tests and documentation for your changes.
|
|
76
77
|
* Test functionality with RSpec; Test features / UI with Capybara.
|
|
77
78
|
* Run _all_ the tests to assure nothing else was accidentally broken.
|
|
78
79
|
|
|
80
|
+
NOTE: This repository follows the [Samvera Community Code of Conduct](https://samvera.atlassian.net/wiki/spaces/samvera/pages/405212316/Code+of+Conduct)
|
|
81
|
+
and [language recommendations](#language).
|
|
82
|
+
Please ***do not*** create a branch called `master` for this repository or as part of
|
|
83
|
+
your pull request; the branch will either need to be removed or renamed before it can
|
|
84
|
+
be considered for inclusion in the code base and history of this repository.
|
|
85
|
+
|
|
79
86
|
### Documenting Code
|
|
80
87
|
|
|
81
88
|
* All new public methods, modules, and classes should include inline documentation in [YARD](http://yardoc.org/).
|
|
@@ -131,15 +138,15 @@ further details.
|
|
|
131
138
|
### Submitting Changes
|
|
132
139
|
|
|
133
140
|
* Read the article ["Using Pull Requests"](https://help.github.com/articles/using-pull-requests) on GitHub.
|
|
134
|
-
* Make sure your branch is up to date with its parent branch (i.e.
|
|
135
|
-
* `git checkout
|
|
141
|
+
* Make sure your branch is up to date with its parent branch (i.e. main)
|
|
142
|
+
* `git checkout main`
|
|
136
143
|
* `git pull --rebase`
|
|
137
144
|
* `git checkout <your-branch>`
|
|
138
|
-
* `git rebase
|
|
145
|
+
* `git rebase main`
|
|
139
146
|
* It is a good idea to run your tests again.
|
|
140
147
|
* If you've made more than one commit take a moment to consider whether squashing commits together would help improve their logical grouping.
|
|
141
148
|
* [Detailed Walkthrough of One Pull Request per Commit](http://ndlib.github.io/practices/one-commit-per-pull-request/)
|
|
142
|
-
* `git rebase --interactive
|
|
149
|
+
* `git rebase --interactive main` ([See Github help](https://help.github.com/articles/interactive-rebase))
|
|
143
150
|
* Squashing your branch's changes into one commit is "good form" and helps the person merging your request to see everything that is going on.
|
|
144
151
|
* Push your changes to a topic branch in your fork of the repository.
|
|
145
152
|
* Submit a pull request from your fork to the project.
|
|
@@ -149,12 +156,14 @@ further details.
|
|
|
149
156
|
We adopted [Github's Pull Request Review](https://help.github.com/articles/about-pull-request-reviews/) for our repositories.
|
|
150
157
|
Common checks that may occur in our repositories:
|
|
151
158
|
|
|
152
|
-
1.
|
|
153
|
-
2.
|
|
159
|
+
1. [CircleCI](https://circleci.com/gh/samvera) - where our automated tests are running
|
|
160
|
+
2. RuboCop/Bixby - where we check for style violations
|
|
161
|
+
3. Approval Required - Github enforces at least one person approve a pull request. Also, all reviewers that have chimed in must approve.
|
|
162
|
+
4. CodeClimate - is our code remaining healthy (at least according to static code analysis)
|
|
154
163
|
|
|
155
164
|
If one or more of the required checks failed (or are incomplete), the code should not be merged (and the UI will not allow it). If all of the checks have passed, then anyone on the project (including the pull request submitter) may merge the code.
|
|
156
165
|
|
|
157
|
-
*Example: Carolyn submits a pull request, Justin reviews the pull request and approves. However, Justin is still waiting on other checks (
|
|
166
|
+
*Example: Carolyn submits a pull request, Justin reviews the pull request and approves. However, Justin is still waiting on other checks (CI tests are usually the culprit), so he does not merge the pull request. Eventually, all of the checks pass. At this point, Carolyn or anyone else may merge the pull request.*
|
|
158
167
|
|
|
159
168
|
#### Things to Consider When Reviewing
|
|
160
169
|
|
|
@@ -171,7 +180,7 @@ This is your chance for a mentoring moment of another developer. Take time to gi
|
|
|
171
180
|
* Do new or changed methods, modules, and classes have documentation?
|
|
172
181
|
* Does the commit contain more than it should? Are two separate concerns being addressed in one commit?
|
|
173
182
|
* Does the description of the new/changed specs match your understanding of what the spec is doing?
|
|
174
|
-
* Did the
|
|
183
|
+
* Did the Continuous Integration tests complete successfully?
|
|
175
184
|
|
|
176
185
|
If you are uncertain, bring other contributors into the conversation by assigning them as a reviewer.
|
|
177
186
|
|
data/README.md
CHANGED
|
@@ -1,10 +1,18 @@
|
|
|
1
1
|
# Noid::Rails
|
|
2
2
|
|
|
3
|
-
Code:
|
|
3
|
+
Code:
|
|
4
|
+
[](http://badge.fury.io/rb/noid-rails)
|
|
5
|
+
[](https://circleci.com/gh/samvera/noid-rails)
|
|
6
|
+
[](https://coveralls.io/github/samvera/noid-rails?branch=main)
|
|
7
|
+
[](https://codeclimate.com/github/samvera/noid-rails/maintainability)
|
|
4
8
|
|
|
5
|
-
Docs:
|
|
9
|
+
Docs:
|
|
10
|
+
[](./CONTRIBUTING.md)
|
|
11
|
+
[](./LICENSE)
|
|
12
|
+
[](http://rubydoc.info/gems/noid-rails)
|
|
13
|
+
[](https://inch-ci.org/github/samvera/noid-rails)
|
|
6
14
|
|
|
7
|
-
|
|
15
|
+
Community Support: [](http://slack.samvera.org/)
|
|
8
16
|
|
|
9
17
|
# What is noid-rails?
|
|
10
18
|
|
|
@@ -14,11 +22,13 @@ This gem mints identifiers for models in your Rails-based application with opaqu
|
|
|
14
22
|
|
|
15
23
|
## Product Owner & Maintenance
|
|
16
24
|
|
|
17
|
-
|
|
25
|
+
`noid-rails` was a Core Component of the Samvera Community. Given a decline in available labor required for maintenance, this project no longer has a dedicated Product Owner. The documentation for what this means can be found [here](http://samvera.github.io/core_components.html#requirements-for-a-core-component).
|
|
18
26
|
|
|
19
27
|
### Product Owner
|
|
20
28
|
|
|
21
|
-
|
|
29
|
+
**Vacant**
|
|
30
|
+
|
|
31
|
+
_Until a Product Owner has been identified, we ask that you please direct all requests for support, bug reports, and general questions to the [`#dev` Channel on the Samvera Slack](https://samvera.slack.com/app_redirect?channel=dev)._
|
|
22
32
|
|
|
23
33
|
## Table of Contents
|
|
24
34
|
|
|
@@ -219,6 +229,12 @@ end
|
|
|
219
229
|
|
|
220
230
|
And the service will delegate minting and validating to an instance of your customized minter class.
|
|
221
231
|
|
|
232
|
+
## Contributing
|
|
233
|
+
|
|
234
|
+
If you're working on PR for this project, create a feature branch off of `main`.
|
|
235
|
+
|
|
236
|
+
This repository follows the [Samvera Community Code of Conduct](https://samvera.atlassian.net/wiki/spaces/samvera/pages/405212316/Code+of+Conduct) and [language recommendations](https://github.com/samvera/maintenance/blob/master/templates/CONTRIBUTING.md#language). Please ***do not*** create a branch called `master` for this repository or as part of your pull request; the branch will either need to be removed or renamed before it can be considered for inclusion in the code base and history of this repository.
|
|
237
|
+
|
|
222
238
|
## Releasing
|
|
223
239
|
|
|
224
240
|
1. `bundle install`
|
data/lib/noid/rails/engine.rb
CHANGED
|
@@ -5,6 +5,18 @@ require 'rails'
|
|
|
5
5
|
module Noid
|
|
6
6
|
module Rails
|
|
7
7
|
class Engine < ::Rails::Engine
|
|
8
|
+
config.before_configuration do
|
|
9
|
+
# rubocop:disable Style/IfUnlessModifier
|
|
10
|
+
|
|
11
|
+
# see https://github.com/fxn/zeitwerk#for_gem
|
|
12
|
+
# Blacklight puts a generator into LOCAL APP lib/generators, so tell
|
|
13
|
+
# zeitwerk to ignore the whole directory? If we're using zeitwerk
|
|
14
|
+
#
|
|
15
|
+
# See: https://github.com/cbeer/engine_cart/issues/117
|
|
16
|
+
if ::Rails.try(:autoloaders).try(:main).respond_to?(:ignore)
|
|
17
|
+
::Rails.autoloaders.main.ignore(::Rails.root.join('lib', 'generators'))
|
|
18
|
+
end
|
|
19
|
+
end
|
|
8
20
|
end
|
|
9
21
|
end
|
|
10
22
|
end
|
data/lib/noid/rails/version.rb
CHANGED
data/noid-rails.gemspec
CHANGED
|
@@ -17,12 +17,13 @@ Gem::Specification.new do |spec|
|
|
|
17
17
|
spec.files = `git ls-files -z`.split("\x0")
|
|
18
18
|
spec.test_files = spec.files.grep(%r{^spec/})
|
|
19
19
|
spec.require_paths = ['lib']
|
|
20
|
+
spec.metadata = { "rubygems_mfa_required" => "true" }
|
|
20
21
|
|
|
21
|
-
spec.add_dependency 'actionpack', '>= 5.0.0', '<
|
|
22
|
+
spec.add_dependency 'actionpack', '>= 5.0.0', '< 8'
|
|
22
23
|
spec.add_dependency 'noid', '~> 0.9'
|
|
23
24
|
|
|
24
|
-
spec.add_development_dependency 'bixby', '~>
|
|
25
|
-
spec.add_development_dependency 'bundler', '>= 1
|
|
25
|
+
spec.add_development_dependency 'bixby', '~> 5.0.0'
|
|
26
|
+
spec.add_development_dependency 'bundler', '>= 2.1'
|
|
26
27
|
spec.add_development_dependency 'engine_cart', '~> 2.2'
|
|
27
28
|
spec.add_development_dependency 'rake', '>= 11'
|
|
28
29
|
spec.add_development_dependency 'rspec', '~> 3.8'
|
metadata
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: noid-rails
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 3.0
|
|
4
|
+
version: 3.2.0
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- Michael J. Giarlo
|
|
8
8
|
autorequire:
|
|
9
9
|
bindir: bin
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date:
|
|
11
|
+
date: 2024-09-06 00:00:00.000000000 Z
|
|
12
12
|
dependencies:
|
|
13
13
|
- !ruby/object:Gem::Dependency
|
|
14
14
|
name: actionpack
|
|
@@ -19,7 +19,7 @@ dependencies:
|
|
|
19
19
|
version: 5.0.0
|
|
20
20
|
- - "<"
|
|
21
21
|
- !ruby/object:Gem::Version
|
|
22
|
-
version: '
|
|
22
|
+
version: '8'
|
|
23
23
|
type: :runtime
|
|
24
24
|
prerelease: false
|
|
25
25
|
version_requirements: !ruby/object:Gem::Requirement
|
|
@@ -29,7 +29,7 @@ dependencies:
|
|
|
29
29
|
version: 5.0.0
|
|
30
30
|
- - "<"
|
|
31
31
|
- !ruby/object:Gem::Version
|
|
32
|
-
version: '
|
|
32
|
+
version: '8'
|
|
33
33
|
- !ruby/object:Gem::Dependency
|
|
34
34
|
name: noid
|
|
35
35
|
requirement: !ruby/object:Gem::Requirement
|
|
@@ -50,28 +50,28 @@ dependencies:
|
|
|
50
50
|
requirements:
|
|
51
51
|
- - "~>"
|
|
52
52
|
- !ruby/object:Gem::Version
|
|
53
|
-
version:
|
|
53
|
+
version: 5.0.0
|
|
54
54
|
type: :development
|
|
55
55
|
prerelease: false
|
|
56
56
|
version_requirements: !ruby/object:Gem::Requirement
|
|
57
57
|
requirements:
|
|
58
58
|
- - "~>"
|
|
59
59
|
- !ruby/object:Gem::Version
|
|
60
|
-
version:
|
|
60
|
+
version: 5.0.0
|
|
61
61
|
- !ruby/object:Gem::Dependency
|
|
62
62
|
name: bundler
|
|
63
63
|
requirement: !ruby/object:Gem::Requirement
|
|
64
64
|
requirements:
|
|
65
65
|
- - ">="
|
|
66
66
|
- !ruby/object:Gem::Version
|
|
67
|
-
version: '1
|
|
67
|
+
version: '2.1'
|
|
68
68
|
type: :development
|
|
69
69
|
prerelease: false
|
|
70
70
|
version_requirements: !ruby/object:Gem::Requirement
|
|
71
71
|
requirements:
|
|
72
72
|
- - ">="
|
|
73
73
|
- !ruby/object:Gem::Version
|
|
74
|
-
version: '1
|
|
74
|
+
version: '2.1'
|
|
75
75
|
- !ruby/object:Gem::Dependency
|
|
76
76
|
name: engine_cart
|
|
77
77
|
requirement: !ruby/object:Gem::Requirement
|
|
@@ -194,7 +194,8 @@ files:
|
|
|
194
194
|
homepage: https://github.com/samvera/noid-rails
|
|
195
195
|
licenses:
|
|
196
196
|
- Apache2
|
|
197
|
-
metadata:
|
|
197
|
+
metadata:
|
|
198
|
+
rubygems_mfa_required: 'true'
|
|
198
199
|
post_install_message:
|
|
199
200
|
rdoc_options: []
|
|
200
201
|
require_paths:
|
|
@@ -210,7 +211,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
|
210
211
|
- !ruby/object:Gem::Version
|
|
211
212
|
version: '0'
|
|
212
213
|
requirements: []
|
|
213
|
-
rubygems_version: 3.
|
|
214
|
+
rubygems_version: 3.0.8
|
|
214
215
|
signing_key:
|
|
215
216
|
specification_version: 4
|
|
216
217
|
summary: Noid identifier services for Rails-based applications
|