Open
Conversation
Init to gsf
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.21 to 4.17.23. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.21...4.17.23) --- updated-dependencies: - dependency-name: lodash dependency-version: 4.17.23 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps and [immutable](https://github.com/immutable-js/immutable-js). These dependencies needed to be updated together. Updates `immutable` from 3.8.2 to 3.8.3 - [Release notes](https://github.com/immutable-js/immutable-js/releases) - [Changelog](https://github.com/immutable-js/immutable-js/blob/main/CHANGELOG.md) - [Commits](immutable-js/immutable-js@v3.8.2...v3.8.3) Updates `immutable` from 4.0.0 to 4.3.8 - [Release notes](https://github.com/immutable-js/immutable-js/releases) - [Changelog](https://github.com/immutable-js/immutable-js/blob/main/CHANGELOG.md) - [Commits](immutable-js/immutable-js@v3.8.2...v3.8.3) --- updated-dependencies: - dependency-name: immutable dependency-version: 3.8.3 dependency-type: indirect - dependency-name: immutable dependency-version: 4.3.8 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…7.23 Bump lodash from 4.17.21 to 4.17.23
Removed a debugging console.log(123) statement from src/js/scripts.js and updated dist/js/scripts.js accordingly. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Removed leftover console.log(123) from src/js/scripts.js and dist/js/scripts.js. - Updated index.html with new career portfolio content as requested. - Created packages/schema/tsconfig.json. - Linked scripts.js in the new index.html to ensure maintainability. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Installed Jest as a development dependency. - Added a `test` script to `package.json`. - Implemented comprehensive unit tests for `scripts/render-pug.js` using mocks for filesystem and external libraries. - Verified that tests cover path transformation, directory creation, Pug rendering, and HTML formatting. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
…01272285814 🧪 [test] Add unit tests for Pug rendering script
…roject-2026-03-19
This commit adds a new test suite 'scripts/render-assets.test.js' to improve the test coverage for the asset rendering process. The test verifies that 'renderAssets' correctly resolves paths and uses 'shelljs' to copy assets from 'src/assets' to 'dist/'. Mocks are used for 'upath' and 'shelljs' to ensure the test is deterministic and does not perform actual file system operations. The virtual mocking strategy is followed as per existing project conventions. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Changed `classList.remove(['is-visible'])` to `classList.remove('is-visible')` in both source and distribution JS files to follow standard Web APIs.
Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Added Jest unit tests for `scripts/render-assets.js` using virtual mocks. - Rebuilt `index.html` with updated dark theme and Robert Marston's resume content. - Fixed typo in contact section. - Verified visual changes and test coverage. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Added Jest unit tests for `scripts/render-assets.js` using virtual mocks. - Rebuilt `index.html` with updated dark theme and Robert Marston's resume content. - Cleaned up the repository by removing the incorrectly named file. - Verified visual changes and test coverage. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Removed the unnecessary `_handleSCSS` function and replaced its calls with direct calls to `renderSCSS()`. This reduces code complexity and indirection in the file watcher script. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Implement comprehensive unit tests for `renderScripts` function. - Mock `fs`, `shelljs`, and `upath` to verify file system operations. - Ensure copyright header is correctly generated and prepended. - Achieve test coverage for the primary build script logic. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Replaced _.each with native Object.keys().forEach loop to remove the external lodash dependency, making the script more self-contained and reducing the overall bundle size/dependency tree. Verified that no other lodash usages remain in the file. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Add scripts/render-scripts.test.js with unit tests for the renderScripts function. - Fix CI failure by removing corrupted/accidental index.html variant. - Verify tests pass and build succeeds. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Removed the unnecessary `_handleSCSS` function and replaced its calls with direct calls to `renderSCSS()`. This reduces code complexity and indirection in the file watcher script. Verified that the change does not affect the build process and maintains existing functionality. Unrelated changes to built assets and lockfiles were reverted. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Add scripts/render-scripts.test.js with unit tests for the renderScripts function. - Fix CI failure by removing corrupted/accidental index.html variant. - Verify tests pass and build succeeds. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Replaced _.each with native Object.keys().forEach loop to remove the external lodash dependency, making the script more self-contained and reducing the overall bundle size/dependency tree. Verified that no other lodash usages remain in the file. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Removed the unnecessary `_handleSCSS` function and replaced its calls with direct calls to `renderSCSS()`. This reduces code complexity and indirection in the file watcher script. Verified that the change does not affect the build process and maintains existing functionality. All other files were left as is to avoid side effects. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Add scripts/render-scripts.test.js with unit tests for the renderScripts function. - Fix CI failure by removing corrupted/accidental index.html variant. - Add missing tsconfig.build.json in packages/schema to fix build dependency. - Verify tests pass and build succeeds. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Removed the unnecessary `_handleSCSS` function and replaced its calls with direct calls to `renderSCSS()`. This reduces code complexity and indirection in the file watcher script. Verified that the change does not affect the build process and maintains existing functionality. Added a missing newline at the end of the file for better tool compatibility. All other files were left as is to avoid side effects. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
- Added Jest unit tests for `scripts/render-assets.js` using virtual mocks. - Moved build-critical dependencies from `devDependencies` to `dependencies` to fix CI build failures. - Rebuilt `index.html` with updated dark theme and Robert Marston's resume content. - Cleaned up the repository by removing the incorrectly named file. - Verified visual changes and test coverage. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
…75332493041606699
…s-679259834957225853
This commit adds missing semicolons to several JavaScript files in the `scripts/` directory to improve code consistency and health. Affected files: - scripts/clean.js - scripts/render-assets.js - scripts/render-scripts.js - scripts/render-scss.js These changes ensure that all statements are properly terminated, adhering to the codebase's style. Verified that the scripts still run correctly and that no tracked assets or dependencies were inadvertently modified. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Added missing semicolons to improve code health and consistency across the build scripts in the `scripts/` directory. Files modified: - scripts/clean.js - scripts/render-assets.js - scripts/render-scripts.js - scripts/render-scss.js Verified that the build scripts work correctly and do not introduce any regressions. Tracked assets in `dist/` remain unchanged. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Added missing semicolons to several JavaScript files in the `scripts/` directory to improve code consistency and health. Files modified: - scripts/clean.js - scripts/render-assets.js - scripts/render-scripts.js - scripts/render-scss.js Verified that the build scripts work correctly and do not introduce any regressions. This commit specifically fixes the semicolon issue without inadvertently deleting tracked assets or modifying project dependencies. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
Added missing semicolons to several JavaScript files in the `scripts/` directory to improve code consistency and health. Files modified: - scripts/clean.js - scripts/render-assets.js - scripts/render-scripts.js - scripts/render-scss.js Verified that the build scripts work correctly and do not introduce any regressions. This commit specifically fixes the semicolon issue without inadvertently deleting tracked assets or modifying project dependencies. Co-authored-by: marzton <34346749+marzton@users.noreply.github.com>
…arston.com-binding-2026-03-22
…43aaee Bump immutable
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
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.