MLE-24733 Deleting what should be unused etc/data files#1025
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes unused data files from the etc/data/ directory as part of cleanup efforts. The removal includes various test data files across different formats (XML, JSON, XSLT, JavaScript, and template definition files) that are no longer needed in the codebase.
- Removes multiple test data files including XML documents, JSON schemas, XSLT stylesheets, and template definition files
- Updates a comment reference in the test file to reflect the removal of the employees.tdej file
- Cleans up validation, transformation, and template files that were previously used for testing
Reviewed Changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| test-basic/rows.js | Removes outdated comment referencing deleted employees.tdej file |
| etc/data/validateDoc-test.sch-validator.xsl | Deletes XSLT validator file for document validation tests |
| etc/data/validateDoc-test.sch | Removes Schematron schema file for validation testing |
| etc/data/validateDoc-test.json | Deletes JSON schema file used for validation tests |
| etc/data/unnestView.tdej | Removes template definition file for unnest view operations |
| etc/data/tripleSets.xml | Deletes XML file containing semantic triple sets for testing |
| etc/data/transformDoc-test.xslt | Removes XSLT transformation file for document testing |
| etc/data/transformDoc-test.mjs | Deletes JavaScript module for document transformation tests |
| etc/data/transformDoc-test-two-params.mjs | Removes JavaScript module with two-parameter transformation |
| etc/data/rowPostProcessors.sjs | Deletes server-side JavaScript file for row post-processing |
| etc/data/musician.tdex | Removes XML template definition for musician data |
| etc/data/masterDetail.xml | Deletes XML file containing master-detail test data |
| etc/data/masterDetail.tdex | Removes XML template definition for master-detail views |
| etc/data/employees.tdej | Deletes JSON template definition for employee data |
| etc/data/articleCitation.json | Removes JSON template for article citation data |
| etc/data/Publisher-2.xml | Deletes second publisher XML test data file |
| etc/data/Publisher-1.xml | Removes first publisher XML test data file |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Copyright Validation Results ✅ Valid Files
✅ All files have valid copyright headers! |
This file contains hidden or 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
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.