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

Sort #105

Merged
merged 3 commits into from
Oct 26, 2023
Merged

Sort #105

merged 3 commits into from
Oct 26, 2023

Conversation

nvolk
Copy link
Contributor

@nvolk nvolk commented Oct 24, 2023

Minor update:

  • update dependencies (mainly marc-record-js and marc-record-validators-melinda)
  • postprocessor: apply new validator for sorting final record
  • fix tests to handle sorting

@nvolk nvolk requested a review from ammsalme October 24, 2023 11:23
@nvolk nvolk requested a review from a team as a code owner October 24, 2023 11:23
@ammsalme ammsalme merged commit 4f736d7 into next Oct 26, 2023
@ammsalme ammsalme deleted the sort branch October 26, 2023 12:29
ammsalme added a commit that referenced this pull request Nov 3, 2023
…pdate (#104)

Update field sorting (#105)
* Update dependencies
* Sort fields in postprocessor and fix affected tests

Pair http and https [MET-501] (#106)
* New feature: http vs https difference in subfield value does not prevent field merge [MET-501]

Dependency updates:
Bump actions/setup-node from 3 to 4 (#102)
Bump the production-dependencies group with 1 update (#103)
Bump the development-dependencies group with 1 update (#101)
Bump the production-dependencies group with 1 update (#108)
Bump the development-dependencies group with 1 update (#109)
Update deps

2.0.19-alpha.4

---------
Co-authored-by: nvolk <[email protected]>
ammsalme pushed a commit that referenced this pull request Nov 8, 2023
Bump actions/setup-node from 3 to 4 (#102)

Bump the production-dependencies group with 1 update (#103)
* Bumps the production-dependencies group with 1 update: [@natlibfi/marc-record-validators-melinda]

Bump the development-dependencies group with 1 update (#101)
* Bumps the development-dependencies group with 1 update: [eslint](https://github.com/eslint/eslint).

Update field sorting (#105)
* Update dependencies
* Sort fields in postprocessor and fix affected tests

Pair http and https [MET-501] (#106)
* New feature: http vs https difference in subfield value does not prevent field merge [MET-501]

* 2.0.20-alpha.2
ammsalme added a commit that referenced this pull request Nov 9, 2023
245punc (#107)

Bump actions/setup-node from 3 to 4 (#102)

Bump the production-dependencies group with 1 update (#103)
* Bumps the production-dependencies group with 1 update: [@natlibfi/marc-record-validators-melinda]

Bump the development-dependencies group with 1 update (#101)
* Bumps the development-dependencies group with 1 update: [eslint](https://github.com/eslint/eslint).

Update field sorting (#105)
* Update dependencies
* Sort fields in postprocessor and fix affected tests

Pair http and https [MET-501] (#106)
* New feature: http vs https difference in subfield value does not prevent field merge [MET-501]

Met502 kustantajat (#111)
* Merge publishers with differing name forms [MET-502]

General
* Update deps
* 2.0.20-alpha.4

---------

Co-authored-by: nvolk <[email protected]>
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