Skip to content

Conversation

@BillFarber
Copy link
Contributor

Also updated the notice file, though I'm not sure if this is required to go through legal first.

Also, added the testRuntimeOnly to build.gradle to prevent the Gradle 9 deprecations.

This comment was marked as outdated.

@BillFarber BillFarber force-pushed the task/updateCopyright branch from b100f31 to 7da1810 Compare August 8, 2025 01:21
@BillFarber BillFarber requested a review from Copilot August 8, 2025 01:22
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the copyright headers across the codebase from "MarkLogic Corporation" to "Progress Software Corporation and/or its subsidiaries or affiliates" for the years 2019-2025, reflecting changes in corporate ownership. Additionally, it includes build.gradle updates to prevent Gradle 9 deprecations and updates the NOTICE file with newer dependency versions.

Key changes include:

  • Updated copyright headers across all Java source files
  • Modified NOTICE.txt with updated copyright and newer dependency versions
  • Updated LICENSE.txt with new copyright information

Reviewed Changes

Copilot reviewed 68 out of 69 changed files in this pull request and generated 2 comments.

File Description
All Java source files Updated copyright headers from MarkLogic Corporation to Progress Software Corporation
NOTICE.txt Updated copyright notice and dependency versions
LICENSE.txt Updated copyright information

* Also verifies that no error is
* thrown. The config option is expected to be ignored because the v1/rows
* endpoint does not support returning
* column types when CSV is requested.
Copy link

Copilot AI Aug 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The comment reformatting has broken the sentence flow awkwardly. Consider keeping the original formatting for better readability.

Suggested change
* column types when CSV is requested.
* Also verifies that no error is thrown. The config option is expected to be ignored because the v1/rows
* endpoint does not support returning column types when CSV is requested.

Copilot uses AI. Check for mistakes.
// While a Transformer is not thread-safe and must therefore be created for each
// batch - though we could consider
// a pooling strategy in the future - a TransformerFactory is thread-safe and
// can thus be reused
Copy link

Copilot AI Aug 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The comment reformatting has broken a coherent thought across multiple lines unnecessarily. The original formatting was likely more readable.

Suggested change
// can thus be reused
// While a Transformer is not thread-safe and must therefore be created for each batch (though we could consider a pooling strategy in the future),
// a TransformerFactory is thread-safe and can thus be reused.

Copilot uses AI. Check for mistakes.
Also updated the notice file, though I'm not sure if this is required to go through legal first.

Also, added the testRuntimeOnly to build.gradle to prevent the Gradle 9 deprecations.
@BillFarber BillFarber force-pushed the task/updateCopyright branch from 7da1810 to 276f5f6 Compare August 8, 2025 01:26
@BillFarber BillFarber closed this Aug 8, 2025
@BillFarber
Copy link
Contributor Author

Too many formatting screw-ups from VS Code.

@BillFarber BillFarber deleted the task/updateCopyright branch August 8, 2025 14:47
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