Skip to content

Conversation

@schloerke
Copy link
Collaborator

@schloerke schloerke commented Jul 6, 2025

Summary from Claude

Key Changes:

Build System Migration:

  • Replaced Yarn configuration (.yarnrc.yml, yarn plugins) with npm
    (package-lock.json)
  • Updated ESLint config from .eslintrc.yml to eslint.config.mjs
  • Removed yarn-specific files and build scripts
  • Updated build tools and dependencies

JavaScript/TypeScript Updates:

  • Fixed circular dependency issues (4-part commit series)
  • Updated PostCSS compilation process
  • Modernized package versions with .x version ranges
  • Removed outdated Jest POC integration and unused packages

Configuration Updates:

  • Updated GitHub Actions workflow to use npm instead of yarn
  • Modified .gitignore and .Rbuildignore files
  • Fixed Sass warnings about deprecated global functions
  • Updated Node.js version references

Code Quality:

  • Applied linting fixes across TypeScript source files
  • Autoformatted code to maintain consistency
  • Built and updated type definitions

The migration successfully maintains all existing functionality while
modernizing the JavaScript build infrastructure and removing outdated
dependencies.

schloerke and others added 30 commits February 3, 2025 13:01
* main: (32 commits)
  Revert the addition of spacing between `icon` and `label` in `actionButton()` (#4248)
  Restrict `icon`/`label` separator spacing to `actionButton()` (#4247)
  Fix front-end action button label updating logic (#4242)
  Update news
  Properly handle undefined value for input subscribe callback (#4243)
  Start new version (#4241)
  v1.11.0 release candidate (#4232)
  Follow up to #3996: fix front-end checkbox label updating logic (#4238)
  feat(InputBinding): subscribe callback now supports event priority (#4211)
  Follow up to #3996 when label is unspecified (i.e., NULL), don't include it in the message (#4237)
  Run routine (#4234)
  chore: #4175 update jquery-ui to 1.14.1 (#4205)
  Update jQuery to 3.7.1 (#3969)
  Fix 404 in example 08_html (shiny.min.css) (#4221)
  Follow up to #3870: fix location of news item (#4233)
  Bugfix for error found in tests (#3870)
  Allow update input labels with HTML (#3996)
  Adds mirai to documentation (#4230)
  family->given for R Core authorship (#4222)
  fix(renderPlot): get interactive plotting working with ggplot2 v4.0 (#4228)
  ...
* main:
  Increment version number to 1.11.1.9000 (#4253)
  v1.11.1 release candidate (#4245)
> ▲ [WARNING] Global built-in functions are deprecated and will be removed in Dart Sass 3.0.0.
> Use meta.mixin-exists instead.
>
> More info and automated migrator: https://sass-lang.com/d/import [plugin sass-plugin]
@schloerke schloerke changed the title chore(js): Migrate from yarn to npm build(js): Migrate from yarn to npm Aug 4, 2025
@schloerke schloerke merged commit 7dcb54b into main Aug 4, 2025
12 checks passed
@schloerke schloerke deleted the yarn_to_npm branch August 4, 2025 20:07
schloerke added a commit that referenced this pull request Aug 5, 2025
Removed references to .prettierrc.yml, jest.config.js, .browserslistrc, and .eslintrc.yml. Added eslint.config.mjs and .claude to the ignore list to reflect current project configuration.

Followup to #4252
schloerke added a commit that referenced this pull request Aug 14, 2025
We compile from R sass. So we should be partial to R's compiler, no the latest in npm
gadenbuie pushed a commit that referenced this pull request Aug 14, 2025
…4272)

* Revert changes in 8861645 from #4252

We compile from R sass. So we should be partial to R's compiler, no the latest in npm

* Restore original compile script (with message) that built using R-sass, not npm

* Add step in build command to update the sass output

* run new command
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.

2 participants