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.

Sign up for GitHub

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

Jump to bottom

Nv20230320 #33

Merged
merged 36 commits into from
Mar 24, 2023
Merged

Nv20230320 #33

merged 36 commits into from
Mar 24, 2023

Conversation

nvolk
Copy link
Contributor

@nvolk nvolk commented Mar 23, 2023

  • support validators/fixes, drop local code
  • improved $9 handling: added support, fixed bugs in & support
  • Fix MRA-490
  • Fix MRA-487
  • Raise Node version from >=14 to >=16 (waypoint on the way 18)
  • add tests for the above things and other stuff

nvolk added 29 commits March 20, 2023 11:38
@nvolk
Update validators/fixers to latest version
dcbe3a7
@nvolk
Update dependencies
dc1b551
@nvolk
Update test to reflect improvements in subfield $0 validator/fixer.
3db0bad
@nvolk
new subfield $6 test
bc0ab5a
@nvolk
New test file ($9 WHATEVER <TRANS> mergability)
df1515d
@nvolk
Set "only" to false
d74c0a7
@nvolk
update dependencies
bd3a579
@nvolk
Explicitly set "only" to false
02e6e72
@nvolk
Excplicitly set "only" to false
5c97ccb
@nvolk
New test file ($9 FOO <TRANS> tests)
554b1ee
@nvolk
Log less
f2471c6
@nvolk
Log less
833d527
@nvolk
Modify logging
b610b28
@nvolk
log less
0d3f7bf
@nvolk
Rewrite $9 handling: Add <TRANS> support. Rewrite of/bug fixes for <K…
99c38e9 
…EEP> and <DROP> support.
@nvolk
Update dependencies
00f00c0
@nvolk
Fix log message
c7c5afb
@nvolk
New file for testing MRA-490
333b3e1
@nvolk
Fix MRA-490
942d191
@nvolk
Test also MRA-487
1fca9a0
@nvolk
Add field 774 related test (don't merge two totally unrelated fields)
a200a16
@nvolk
Field-merge: list more fields, that ignore indicator(s) when making m…
6d237ac 
…erge decision
@nvolk
field-merge: remove some blocked tags
5298259
@nvolk
Make contraints way stricter for 760-788 fields
124ff99
@nvolk
Tidy up
9e46409
@nvolk
Remove obsolete commented code
fc180ab
@nvolk
Node 14 => 16 etc
4e3e7c6
@nvolk
Mode version from >=14 to >=16
c9eaaec
@nvolk
Update dependencies
dcdd1a4
@nvolk nvolk requested a review from ammsalme March 23, 2023 11:46
@nvolk nvolk requested a review from a team as a code owner March 23, 2023 11:46
@nvolk
Copy link
Contributor Author

nvolk commented Mar 24, 2023

Raise node version to 18

@ammsalme ammsalme merged commit 96df16c into dev Mar 24, 2023
ammsalme added a commit that referenced this pull request Mar 27, 2023
@ammsalme @nvolk
Updates and maintenance update (#33) (#34)
f69c820 
* Node v14 -> v18

*  Update marc-record-validators-melinda: v1.2.3 (https://github.com/NatLibFi/marc-record-validators-melinda/tree/v10.2.3)

* Update dependencies

* Updates to handling subfield $9
** Add <TRANS> support. 
** Rewrite of/bug fixes for <KEEP> and <DROP> support.

* Updates to handling 7xx linking (f766-f780) fields (MRA-487)

* Updates to avoid duplicating f300 subfield $a (MRA-490)

* Reduce and modify logging

* 2.0.0-alpha.1

Co-authored-by: nvolk <[email protected]>
@ammsalme ammsalme deleted the nv20230320 branch March 27, 2023 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ammsalme ammsalme ammsalme approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nvolk @ammsalme