Sonar dependency update, round 2.

Review Request #12447 — Created July 8, 2022 and submitted

Information

student-sonar
master

Reviewers

This is the second phase of updating sonar's dependencies. Many of these
ones are focused more on the frontend side of things, including updating
to newer versions of react.

This also switches us over to using the "module" type in package.json.
This tells node that we're operating using ES6 modules rather than
CommonJS imports. This eliminates the need to use babel-node, which
simplifies things considerably and should improve performance a lot. The
one casualty of this is the newrelic integration, which would need
external configuration (since apparently the newrelic.js file can't be
loaded in ESM mode). I don't really care about newrelic monitoring here,
so this is fine for now.

I've also deleted the old load-old-data.js script. This was used when
we moved the data storage over from MongoDB to SQL, but isn't necessary
anymore.

Logged in and checked various features. In particular, tested parts of
the UI affected by the selectize update (things like group membership
selectors), and React-MDE (the status report markdown editor).

Summary ID
Sonar dependency update, round 2.
This is the second phase of updating sonar's dependencies. Many of these ones are focused more on the frontend side of things, including updating to newer versions of react. This also switches us over to using the "module" type in `package.json`. This tells node that we're operating using ES6 modules rather than CommonJS imports. This eliminates the need to use babel-node, which simplifies things considerably and should improve performance a lot. The one casualty of this is the newrelic integration, which would need external configuration (since apparently the `newrelic.js` file can't be loaded in ESM mode). I don't really care about newrelic monitoring here, so this is fine for now. I've also deleted the old `load-old-data.js` script. This was used when we moved the data storage over from MongoDB to SQL, but isn't necessary anymore. Testing Done: Logged in and checked various features. In particular, tested parts of the UI affected by the selectize update (things like group membership selectors), and React-MDE (the status report markdown editor).
837babca70e1fe9bb00577cbc18668f4667868e0
Description From Last Updated

'fs' is defined but never used. Column: 8 Error code: W098

reviewbotreviewbot

'shell' is defined but never used. Column: 8 Error code: W098

reviewbotreviewbot

import.meta may only be used in module code. Column: 40 Error code: E070

reviewbotreviewbot

import.meta may only be used in module code. Column: 40 Error code: E070

reviewbotreviewbot

import.meta may only be used in module code. Column: 40 Error code: E070

reviewbotreviewbot

import.meta may only be used in module code. Column: 40 Error code: E070

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 passed.
JSHint failed.

JSHint

  • gulpfile.js (Diff revision 1)
     
     
    Show all issues
    'fs' is defined but never used.
    
    Column: 8
    Error code: W098
  • gulpfile.js (Diff revision 1)
     
     
    Show all issues
    'shell' is defined but never used.
    
    Column: 8
    Error code: W098
  • lib/api.js (Diff revision 1)
     
     
    Show all issues
    import.meta may only be used in module code.
    
    Column: 40
    Error code: E070
  • webpack.config.js (Diff revision 1)
     
     
    Show all issues
    import.meta may only be used in module code.
    
    Column: 40
    Error code: E070
  • 
      
david
Review request changed
Commits:
Summary ID
Sonar dependency update, round 2.
This is the second phase of updating sonar's dependencies. Many of these ones are focused more on the frontend side of things, including updating to newer versions of react. This also switches us over to using the "module" type in `package.json`. This tells node that we're operating using ES6 modules rather than CommonJS imports. This eliminates the need to use babel-node, which simplifies things considerably and should improve performance a lot. The one casualty of this is the newrelic integration, which would need external configuration (since apparently the `newrelic.js` file can't be loaded in ESM mode). I don't really care about newrelic monitoring here, so this is fine for now. I've also deleted the old `load-old-data.js` script. This was used when we moved the data storage over from MongoDB to SQL, but isn't necessary anymore. Testing Done: Logged in and checked various features. In particular, tested parts of the UI affected by the selectize update (things like group membership selectors), and React-MDE (the status report markdown editor).
c4015367215171554f6139b54f6c4c2a4944b489
Sonar dependency update, round 2.
This is the second phase of updating sonar's dependencies. Many of these ones are focused more on the frontend side of things, including updating to newer versions of react. This also switches us over to using the "module" type in `package.json`. This tells node that we're operating using ES6 modules rather than CommonJS imports. This eliminates the need to use babel-node, which simplifies things considerably and should improve performance a lot. The one casualty of this is the newrelic integration, which would need external configuration (since apparently the `newrelic.js` file can't be loaded in ESM mode). I don't really care about newrelic monitoring here, so this is fine for now. I've also deleted the old `load-old-data.js` script. This was used when we moved the data storage over from MongoDB to SQL, but isn't necessary anymore. Testing Done: Logged in and checked various features. In particular, tested parts of the UI affected by the selectize update (things like group membership selectors), and React-MDE (the status report markdown editor).
837babca70e1fe9bb00577cbc18668f4667868e0

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

  • lib/api.js (Diff revision 2)
     
     
    Show all issues
    import.meta may only be used in module code.
    
    Column: 40
    Error code: E070
  • webpack.config.js (Diff revision 2)
     
     
    Show all issues
    import.meta may only be used in module code.
    
    Column: 40
    Error code: E070
  • 
      
maubin
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to master (0cebb0c)