TAP5-2804: Convert Tapestry JavaScript sources from CoffeeScript to TypeScript #53
Conversation
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
Thorough tests still needed, though.
Thorough tests still needed, though.
since it causes a duplicated variable compilation error in generated AMD/Require.js modules
to tapestry-core to keep things simple (even if a bit less organized)
This reverts commit 5cbfade.
This reverts commit 5b10b01.
This reverts commit d9e150c.
This reverts commit b6822a6.
This reverts commit e02d8e0.
This reverts commit c84c17c.
This reverts commit ab926c9.
This reverts commit fb457f8.
This reverts commit 06c87a8.
This reverts commit a952e27.
This reverts commit e8e10d9.
This reverts commit eb37533.
thiagohp
added a commit
that referenced
this pull request
Jun 14, 2025
#53) * decaffeinate: Rename ajax.coffee and 27 other files from .coffee to .js * decaffeinate: Convert ajax.coffee and 27 other files to JS * TAP5-2804: moving converted CoffeeScript sources to TS project * TAP5-2804: creating TypeScript project * TAP5-2804: changing file extensions from .js to .ts * TAP5-2804: moving TypeScript files to correct folder * TAP5-2805: Upgrade underscore.js to 1.36.7 * TAP5-2804: another pass at the CoffeeScript to TypeScript conversion * TAP5-2804: finishing the CoffeeScript to TypeScript conversion Thorough tests still needed, though. * TAP5-2804: finishing the CoffeeScript to TypeScript conversion Thorough tests still needed, though. * TAP5-2804: adding dom-prototype.ts plus little tweaks * TAP5-2804: adapting the build system to the TS codebase * TAP5-2804: renaming 'exports' variable in TS sources since it causes a duplicated variable compilation error in generated AMD/Require.js modules * TAP5-2804: fixing typeScriptDocs() task type * TAP5-2804: removing t5-core-dom.coffee * decaffeinate: Rename beanvalidator-validation.coffee from .coffee to .js * decaffeinate: Convert beanvalidator-validation.coffee to JS * TAP5-2804: moving former Coffee file to resources * TAP5-2804: moving beanvalidator-validation JS to tapestry-core to keep things simple (even if a bit less organized) * TAP5-2804: beanvalidator-validation JS to TS * TAP5-2804: tapestry-beanvalidator cleanup * decaffeinate: Rename zonedemo.coffee and 10 other files from .coffee to .js * decaffeinate: Convert zonedemo.coffee and 10 other files to JS * TAP5-2804: moving converted CoffeeScript to JS files * TAP5-2804: more CS to TS adjustments * TAP5-2804: removing last remnants of CoffeeScript * Trying to figure out Jenkins test failures * Another attempt to figure out Jenkins test failures * Attempt #3 * TAP5-2804: bunch of small adjustments * TAP5-2804: fixing imports and JSDoc * TAP5-2804: initial work on option to disable Require.js * TAP5-2804: more little fixes * TAP5-2804: fixing onDocument() and its usages, plus imports * TAP5-2804: adapting test JS code to AMD module changes. * TAP5-2804: jQuery fixes * TAP5-2804: test JS fixes plus TS conversion fixes * TAP5-2804: fixing scanners initialization in dom.js * TAP5-2804: fixed ElementWrapper.attr() in jQuery. * Enabling logging to Gradle console on tapestry-beanvalidator * TAP5-2804: task tapestry-core:compileTypeScript not being called * TAP5-2804: renaming Require.js-enabling configuration symbol * Trying to speed up integration tests * TAP5-2804: adding test suites for Require.js enabled and not * Disabling tests with Require.js disabled while it's been worked * Fixing broken tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.