forked from DSpace/DSpace
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Upgrade DSpace to CLARIN-DSpace #1118
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove dtqExampleTest
* Added email validation and input/output encodings are loaded from the cfg. * Fixed checkstyle violation * Refactored condition which check empty character in the email.
* Defined `local.files.count` and `local.files.size` and that values are added into metadata fields during updating of the Item. * Defined downloading zip cfg properties * Exposed download zip cfg properties * Removed commit which starts active session and because of that some tests are failing. * Removed VersioningWithRelationshipsTest.java because it is duplicate of VersioningWithRelationshipsIT and that is also removed from Vanilla * Update Item's file metadata in other place - not in the ItemServiceImpl#update because it is causing errors * Count files info only for the ORIGINAL bundle * Updated StatisticsRestRepositoryIT.java in the way that creating of some bitstreams are done in the setup when the Item is still persisted and not detached * Removed the function from the ItemServiceImpl#update method
fixed wrong number of parameters expected in java code and wrong number of parameters sent in xsl file
* Updated xoia.xml following the actual xoai.xml in v5 * Empty commit to run actions * Fixed path to the ColComFilter in the xoai.xml
* Update Item's metadata when the bitstream is updated. * Update Item's metadata about files after deleting the bitstream. * Update Item's metadata before the bundle is deleted or touched * Check if the bundle is not null before using it.
Some xsl crosswalks call functions that expect parameters, but for various reasons, parameters are empty. That is now checked and logged.
* Added authorization for the `Edit Item - License` menu option. * Created endpoint for updating license of the Item. * Added functionality only for detaching the license.
* The complex input field value could be empty - removed condition that requires complex input data to not be empty. * Fixed issue when the `jsonValEvaluator` is null because of empty operation value. * Updated doc. * Mark inputs for funding as required. Added an exception when the openaire_id is not required. * Updated `submission-forms` complex input field to be not required for Integration tests. * Refactored and updated validation for complex input field
…it should be unbound (#659)
* Added dataProvider Facet * Replaced deprecated `sortOrder` with the new one. * `searchFacetDataProvider` should be in the searchFilters * Updated Integration test which checks the search filters
Add olac description with lindat values
add elg format with appropriate functions
* dso has never been asigned * Use Factory for the Collection and Community service . * Use Handle Service instead of HandleResolver which calls HandleService
* Created method which updates properly the Item's date metadata. * Fixed checkstyle issues * The servlet request could be null. * Update the comment to make description more clear.
Use correct middle part in special Output Stream.
solr log modification with explanation
…em-submission.xml are ignored. (#672) Co-authored-by: Ondřej Košarko <[email protected]>
added cesilko cmdi file with explanatory readme
* [DURACOM-156] Singposting: feedback after initial deployment * Update LICENSES_THIRD_PARTY for 7.6 * [maven-release-plugin] prepare release dspace-7.6 * [maven-release-plugin] prepare for next development iteration * Improve performance of automatic discovery re-index after database changes * Ignore vocabulary indexes without discovery facet to avoid NPE * [maven-release-plugin] prepare branch dspace-7_x * 103837: Only send GA4 events for ORIGINAL bitstreams + ITs * Add a configuration key to disable hierarchical browse indexes * More and better(?) documentation. * Don't clear the list of accepted embedded message header properties. Why was this cleared? * Simplify and modernize the code. Store content directly in the template. * Add flag Pattern.UNICODE_CHARACTER_CLASS to pattern compilation to recognize unicode characters * Enable Pull Request Opened action to assign PRs to their creator * Ensure codescan and label_merge_conflicts run on maintenance branches * Split docker image builds into separate jobs to allow them to run in parallel. * Ensure 'main' code is tagged as 'latest' in DockerHub * Fixes DSpace#8558 - set Solr UTC timezone Set the timezone of the Solr date formatter to UTC * remove obsolete code fragments * Add action to automatically create a port PR when specified (cherry picked from commit f6a898c) * Run PR Port action as 'dspace-bot' to allow new PRs to trigger CI checks * Minor update to label_merge_conflicts to ignore any errors (seem random at this time) * Fix typo. Config must be a valid regex (cherry picked from commit 7995289) * Bump h2 from 2.1.214 to 2.2.220 Bumps [h2](https://github.com/h2database/h2database) from 2.1.214 to 2.2.220. - [Release notes](https://github.com/h2database/h2database/releases) - [Commits](h2database/h2database@version-2.1.214...version-2.2.220) --- updated-dependencies: - dependency-name: com.h2database:h2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> * Update FullTextContentStreams.java Fix NPE if bitstream is null * Update FullTextContentStreams.java Add additional NPE checks * Update ItemUtils.java Prevent npe if bitstream is null * dspace.cfg: remove old webui.itemlist properties These properties are no longer used in DSpace 7: webui.itemlist.widths webui.itemlist.*.widths webui.itemlist.tablewidth (cherry picked from commit 16c46c4) * dspace.cfg: remove old webui.browse.thumbnail.show property The webui.browse.thumbnail.show property is no longer used as of DSpace 7. Also remove subsequent references to adding thumbnails to item view columns since these are from legacy DSpace. (cherry picked from commit 213a546) * dspace.cfg: remove old webui.itemlist.browse.* property The webui.itemlist.browse.* properties are no longer used in DSpace 7. (cherry picked from commit 35f72bc) * Fix DSpace#8963: Remove deletion constraint from Groomer * Enable entity type to submission form mapping by default (cherry picked from commit b71eee8) * [DURACOM-179] replaced 'null' value with exception actual value in sendErrorResponse method calls having 'null' (cherry picked from commit aa35a47) * DS-8935. webui.browse.link CrossLinks - Fix for multiple exact matches Fixes DSpace#8935 when multiple exact match "webui.browse.link" configuration entries are present that point to different indexes. Modified the code to return the index associated with the given metadata (which is used as the key in the hash map), instead of the key from the keySet (which may not actually be the metadata value being searched for). DSpace#8935 (cherry picked from commit b846c53) * DSpace#9006 fix referenced configuration file (cherry picked from commit 29a88d7) * DSpace#9006 fix referenced configuration file (Test) (cherry picked from commit 309b0b3) * Fix DSpace#8933: Only add the base statistic core if it hasn't already been added * Remove duplicate code * On failure log the name of the assetstore file and trace causes of exception. (cherry picked from commit 22974e9) * Report Throwable's type too. (cherry picked from commit d6b612f) * More description on OutOfMemoryError too. (cherry picked from commit bbe5df3) * Remove useless log.info (cherry picked from commit 1f3ad99) * 103837: Refactor GA config to list bundles * README.md: Fix typo (cherry picked from commit ca8abdd) * Bump up versions of buildnumber-maven-plugin & build-helper-maven-plugin. add configuration for SCM failure (cherry picked from commit 78ea9e8) * 3331 - remove the --optimize feature of 'dspace stats-util' (cherry picked from commit 08c6509) * fix MissingOptionException on help (cherry picked from commit 8ae5ffb) * fix stylecheck (cherry picked from commit b1377ca) * ingore unrecognized arguments on help (cherry picked from commit 82c9b6f) * add DSpaceSkipUnknownArgumentsParser as parser to ignore/skip unknown arguments in cli by help, fix not necessary ParseException in help (cherry picked from commit 86285d7) * remove not necessary else (cherry picked from commit 064e2ca) * Enable new skip merge commit feature * dspace-api: remove unnecessary trailing whitespace * 103818 ItemServiceImpl#inheritCollectionDefaultPolicies now clears item READ policies if new parent collection has a default READ policy * 103818 Add boolean parameter to ItemServiceImpl#inheritCollectionDefaultPolicies to decide whether to override item read policies * 103818 Add boolean parameters to ItemServiceImpl methodds to decide whether to override read policies * 103818 Extend ItemServiceTest#testMoveItemToCollectionWithMoreRestrictiveReadPolicy * 104878 Fix error in ItemServiceTest related to inheriting collection policies upon item move * 104878 Adjust ItemServiceTest to expect correct behavior of bundles when item is migrated * Fix failing IT in BulkAccessControlIT * 103837: Fix isContentBitstream() in GoogleAsyncEventListener * Rename test suites that are really integration testing. (cherry picked from commit f66ca33) * Bump org.eclipse.jetty:jetty-xml Bumps [org.eclipse.jetty:jetty-xml](https://github.com/eclipse/jetty.project) from 9.4.51.v20230217 to 9.4.52.v20230823. - [Release notes](https://github.com/eclipse/jetty.project/releases) - [Commits](jetty/jetty.project@jetty-9.4.51.v20230217...jetty-9.4.52.v20230823) --- updated-dependencies: - dependency-name: org.eclipse.jetty:jetty-xml dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> (cherry picked from commit 2bcc0b3) * Add note that rebooting Tomcat required * Add websvc.opensearch.autolink and websvc.opensearch.shortname to exposed REST configuration properties (cherry picked from commit 80b35c9) * Add a "container friendly" log4j2 cfg and output compose dspace log to console (cherry picked from commit 9eefd56) * Fix ClassCastException (Collection cannot be cast to Item) in Handle identifier classes (cherry picked from commit 1271374) * Fix checkstyle. Correct grammar of comment while doing so. (cherry picked from commit ffa2683) * Avoid double slashes in sitemap paths. (cherry picked from commit eae4463) * Remove 'cross join' from count query. Updates "countHandlesByPrefix" to use a query similar to existing "findByPrefix" (cherry picked from commit 14223bd) * fix logical bug when checking if field is controlled authority * remove optimize option from oai import * added authorization check for license bitstream in OAI import * OAI: add support to extract embargo from bitstreams and expose it in OAI metadata (cherry picked from commit db81d75) * ItemUtils.java: added method doc (cherry picked from commit 51e60fb) * ItemUtils.java: improved method to account for multiple embargo policies and select the longest embargo (cherry picked from commit 538be7f) * Refactored access-status to include embargo date based on the DefaultAccessStatusHelper logic (look at primary or first bitstream) (cherry picked from commit 895926f) * Remove unused imports (cherry picked from commit 4bd2cfd) * Fix style issues (cherry picked from commit 724a4ff) * Fix style issues (cherry picked from commit 6e2c8a4) * Add null check (cherry picked from commit 0de4c39) * ItemUtils.java: refactored addEmbargoField (cherry picked from commit 291afa7) * uketd_dc.xsl: also expose access-status if embargo or restricted (cherry picked from commit 4b40872) * DefaultAccessStatusHelper: fix logic to take shortest embargo (cherry picked from commit d17ef09) * Remove currently unused customisation of ItemUtils (cherry picked from commit 490a982) * DefaultAccessStatusHelper: getEmbargoFromItem return null embargo if status than embargo (cherry picked from commit e05e73a) * Additional Item class cast fixes in handle providers DSOs were not properly checked if they were instanceof Item before attempting the cast in HandleIdentifierProvider and VersionedHandleIdentifierProviderWithCanonicalHandles * Remove Oracle script that accidentally made it in via DSpace#8800 (cherry picked from commit 5e04edf) * 8968 - request-a-copy email: non ASCII characters are encoded as HTML character entity references (cherry picked from commit db36d5e) * unused import (cherry picked from commit bf6e042) * 8968 - added custom StringEscapper (cherry picked from commit 103c8ee) * checkstyle (cherry picked from commit 2c2b3b1) * 8968 - implementated using HtmlUtils scaping (cherry picked from commit 090beed) * checkstiye (cherry picked from commit d12fbe2) * Define _version_ * DSpace#8585 Add submitter information to provenance metadata (cherry picked from commit c15ac0e) * 8585 - added provenance to metadata-import and itemImport (cherry picked from commit ea6307d) * dspace-api: fix misaligned comment (cherry picked from commit 4fba787) * oai_openaire.xsl : change resourceTypeGeneral for thesis Thesis are "Literature" resource type (resourceTypeGeneral), not "other research product" ref: openaire/guidelines-literature-repositories#43 (comment) and https://api.openaire.eu/vocabularies/dnet:result_typologies/publication (cherry picked from commit 669ff34) * Bump org.eclipse.jetty:jetty-http Bumps [org.eclipse.jetty:jetty-http](https://github.com/eclipse/jetty.project) from 9.4.52.v20230823 to 9.4.53.v20231009. - [Release notes](https://github.com/eclipse/jetty.project/releases) - [Commits](jetty/jetty.project@jetty-9.4.52.v20230823...jetty-9.4.53.v20231009) --- updated-dependencies: - dependency-name: org.eclipse.jetty:jetty-http dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> * Bump org.json:json from 20230227 to 20231013 in /dspace-api Bumps [org.json:json](https://github.com/douglascrockford/JSON-java) from 20230227 to 20231013. - [Release notes](https://github.com/douglascrockford/JSON-java/releases) - [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md) - [Commits](https://github.com/douglascrockford/JSON-java/commits) --- updated-dependencies: - dependency-name: org.json:json dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> (cherry picked from commit 11a08f1) * Reading localized license file. * Reading localized license file. * Applying commit 6108c98 * quote Pattern for thumbnail resolution constructed from bitstream filename (cherry picked from commit a124807) * check null value of bitstream name before quoting name for regex (cherry picked from commit a9bcc0c) * extract bitstream thumbnail name pattern into own function (cherry picked from commit c5466c2) * Make workflow curation tasks actually work. When curation runs, there was no "current user" and no claimed task, so the code broke when trying to find people to notify about curation failures. (cherry picked from commit a76af35) * Community request: fake EPerson from configuration. (cherry picked from commit bb9e88d) * Correct some documentation. (cherry picked from commit be22790) * Handle missing role. (cherry picked from commit 2e62fa3) * Fix "Site cannot be indexed" error by ignoring ADD/REMOVE events on Site object (cherry picked from commit ef7f02f) * Add a null check when assigning ldap groups Prevent NullReferenceException by checking if the group list is null Fixes DSpace#8920 (cherry picked from commit bb6498e) * 9043 use Templates for compiled XSLT instead of Transformer - use Templates are thread-safe and NOT Transformer (cherry picked from commit 1160341) * Update to newly released XOAI 3.4.0 (cherry picked from commit 160ebbd) * add test and fix (cherry picked from commit 48b0b71) * unset primary bitstream on bitstream service (cherry picked from commit 47ca74b) * adding sql expression to fix deleted primary bitstreams from bundle (cherry picked from commit 8a531ad) * add bundle remove authorization (cherry picked from commit 3255e07) * adding missing bundle REMOVE authorization (cherry picked from commit 4a05600) * add missing head style check (cherry picked from commit caba4bb) * fix style errors (cherry picked from commit 74605f1) * new testDeleteBitstreamAndUnsetPrimaryBitstreamID test for primary bitstream verification (cherry picked from commit e6d108a) * new testDeleteBitstreamAndUnsetPrimaryBitstreamID test for primary bitstream verification (cherry picked from commit ad0d22a) * new testDeleteBitstreamAndUnsetPrimaryBitstreamID remove unnecessary stubs (cherry picked from commit a3e506c) * make comments more clear to understand (cherry picked from commit c0bbd9d) * typo (cherry picked from commit 74cce86) * Add basic pagination to /groups/[uuid]/epersons endpoint (cherry picked from commit 74c7235) * Bug fix. Only use pageSize and offset if >0 (cherry picked from commit 15de2d0) * Add missing pagination test for /groups/[uuid]/epersons (cherry picked from commit 457dd9a) * Add pagination to /groups/[uuid]/subgroups endpoint, along with tests (cherry picked from commit e7c4b9e) * Add basic unit test for new EpersonService methods (cherry picked from commit c000e54) * Minor unit test fix. Use isEqualCollection to compare list with Hibernate results (cherry picked from commit cdb68a6) * Add countAllMembers() with tests. Update tests to use try/catch (cherry picked from commit 58a15b7) * Replace several usages of allMembers() with count methods to avoid performance issues (cherry picked from commit 2c9165a) * Fix bug in logic for determining whether a workflow group will be left empty. Need to check *both* EPerson and subgroup counts. (cherry picked from commit 9832259) * Use join instead of subquery as join seems slightly faster. (cherry picked from commit 9c0bf08) * Address feedback. Initialize HashSet sizes to avoid resizing. Correct comment about indeterminante ordering. (cherry picked from commit f011a5a) * Allow users with write permission to view hidden metadata (cherry picked from commit 65a17d4) * Test modification: allow users with write rights to see hidden metadata (cherry picked from commit df7f6e9) * Add test to check that user with read rights can see hidden metadata (cherry picked from commit 03496c3) * Verify optional message is not missing or a literal "null" value (cherry picked from commit 534ee3a) * Change the database mode to READ_ONLY during the indexing by discovery consumer (IndexEventConsumer) (cherry picked from commit 94822b5) * Add functions to do a manual flush of the db session and call flush before change to READ_ONLY mode to be sure we index the current object (cherry picked from commit c33d3fa) * Flush database changes after switching to READONLY mode (cherry picked from commit 00a6531) * Add test to check retrieving of policies after changing mode to READ_ONLY (cherry picked from commit d19a959) * Change class name to ContextIT and correct a test (cherry picked from commit a556799) * dspace/config: update spider agent list Update list of spider user agents from the COUNTER-Robots project. See: https://github.com/atmire/COUNTER-Robots (cherry picked from commit 7566a79) * removed options to ping search engines when generating sitemaps (cherry picked from commit f8f8806) * XmlWorkflowCuratorServiceImpl: add check to queue task if configured; Curation: remove obsolete code preventing curation running on workflow tasks as DSpace#3157 is now implemented * Return both user and operational LDAP attributes Explicitly request both user and operation attributes for LDAP group search as the default searching does not include operational attributes. This is required to fetch the memberOf attribute when checking LDAP group membership. Fixes DSpace#9151 (cherry picked from commit 56b7cbf) * [DURACOM-200] improvement of checker script (cherry picked from commit 5a7c795) * 108055: isClosed method should use xml configuration (cherry picked from commit d800d80) * [DURACOM-192] Authentication Method related special groups are put in claim set even if a different authentication method is used (cherry picked from commit 6504d74) * [DURACOM-192] Added test (cherry picked from commit fa39251) * 107891: Cache administrator group (cherry picked from commit 1e82ca7) * DURACOM-199 improved test to show bug related to restricted content (cherry picked from commit dac4df9) * DURACOM-199 fix sitemap generator for restricted content and improve performance (cherry picked from commit 6d9ca38) * [Port dspace-7_x] Event consumer for submission config reloading when a collection changes (DSpace#9196) * initialization with refactoring (cherry picked from commit e93dc1c) * also consider SubmissionConfigReaderException (cherry picked from commit 8a04b87) * rename consumer file (cherry picked from commit f6c92a4) * init submission service factory (cherry picked from commit fcc5239) * set submissionconfig config settings by default (cherry picked from commit e343d51) * renaming SubmissionConfigReaderService (cherry picked from commit 9ea7c32) * support for SubmissionConfigService (cherry picked from commit 61389fb) * fixing style errors and renaming submissionConfigService (cherry picked from commit 5f49491) * fixing style errors and unused imports (cherry picked from commit 31d9251) * set default submission event configs (cherry picked from commit 578198c) * adding force indexing action to Consumer (cherry picked from commit b912364) * stylecheck fixes (cherry picked from commit 89e89c3) * undo event.dispatcher.noindex.consumers (cherry picked from commit ae9dc5f) --------- Co-authored-by: Paulo Graça <[email protected]> * Add isNotMemberOf for groups, including unit and integration tests (cherry picked from commit 9d271b2) * Implement searchNonMembers for EPersonService. Add tests to prove it works (and tests for search()). Requires minor bug fix to AbstractHibernateDSODAO to allow for additional OR/AND clauses to be appended. (cherry picked from commit f186dcf) * Add /epersons/search/isNotMemberOf endpoint to REST API along with integration tests (cherry picked from commit 5208a35) * Bug fix to EPersonDAOImpl. Correctly determine if excluded group needs to be preceded by AND or WHERE (cherry picked from commit e5e0eaa) * Updated IIIF Controller IT to text bitstream and bundle exclusions (cherry picked from commit e92b4b7) * Simplified the process of fixing the tests after adding new sidebar facets/search filters and sort options to discover.xml (cherry picked from commit b40ad0d) * 107671: Expose the handle.canonical.prefix to the frontend (cherry picked from commit 6d86e65) * Remove line breaks from default.license because they are being rendered in the frontend (cherry picked from commit 56aae34) * [Port dspace-7_x] subscription email: do not send email if nothing has changed (DSpace#9204) * improved subscriptions email template (cherry picked from commit 6e7b327) * do not send emails without content (cherry picked from commit 926b242) * fixed coding style violations (cherry picked from commit fdacec0) * removed unnecessary isEmpty check as suggested by reviewer (cherry picked from commit 30a837f) * moved null check on indexableObjects in generateBodyMail (cherry picked from commit b43c340) * fixed unhandled IOException (cherry picked from commit 9b3f7b6) * fixed typo in bodyCommunities (cherry picked from commit ac3d02e) * do not use != to compare strings (cherry picked from commit e460183) * fixed improper handling of empty list (cherry picked from commit ac72aae) --------- Co-authored-by: Sascha Szott <[email protected]> * [maven-release-plugin] prepare release dspace-7.6.1 * remove double permissions * remove duplicate code after sync with 7.6.1 from upstream * remove bean duplicates * Use our discovery.xml * Removed duplicities --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Nikita Krivonosov <[email protected]> Co-authored-by: Tim Donohue <[email protected]> Co-authored-by: Toni Prieto <[email protected]> Co-authored-by: Alan Orth <[email protected]> Co-authored-by: Nona Luypaert <[email protected]> Co-authored-by: Mark H. Wood <[email protected]> Co-authored-by: Sean Kalynuk <[email protected]> Co-authored-by: Sascha Szott <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Agustina Martinez <[email protected]> Co-authored-by: Martin Walk <[email protected]> Co-authored-by: corrado lombardi <[email protected]> Co-authored-by: David P. Steelman <[email protected]> Co-authored-by: Christian Bethge <[email protected]> Co-authored-by: max.nuding <[email protected]> Co-authored-by: Max Nuding <[email protected]> Co-authored-by: Christian Clauss <[email protected]> Co-authored-by: Hrafn Malmquist <[email protected]> Co-authored-by: aroman-arvo <[email protected]> Co-authored-by: Christian Bethge <[email protected]> Co-authored-by: Koen Pauwels <[email protected]> Co-authored-by: Mark Cooper <[email protected]> Co-authored-by: nwoodward <[email protected]> Co-authored-by: Kim Shepherd <[email protected]> Co-authored-by: Marie-Hélène Vézina <[email protected]> Co-authored-by: damian <[email protected]> Co-authored-by: Gantner, Florian Klaus <[email protected]> Co-authored-by: wwuck <[email protected]> Co-authored-by: Paulo Graça <[email protected]> Co-authored-by: mohamed eskander <[email protected]> Co-authored-by: Jens Vannerum <[email protected]> Co-authored-by: Luca Giamminonni <[email protected]> Co-authored-by: Andrea Bollini <[email protected]> Co-authored-by: DSpace Bot <[email protected]> Co-authored-by: Michael Spalti <[email protected]> Co-authored-by: Alexandre Vryghem <[email protected]> Co-authored-by: milanmajchrak <[email protected]>
kosarko
added a commit
that referenced
this pull request
Jul 10, 2024
the commit is the merge commit of #1118
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.