Skip to content

Commit

Permalink
Add *.js suffix in relative require statements, remove unnecessary es…
Browse files Browse the repository at this point in the history
…lint-disable directives, rename require variables, see phetsims/chipper#1498
  • Loading branch information
samreid committed Oct 28, 2024
1 parent efc9f20 commit 56535e7
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion github-dashboard/issueReport.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
* @author Agustín Vallejo (PhET Interactive Simulations)
*/

const buildLocal = require( '../../perennial-alias/js/common/buildLocal' );
const buildLocal = require( '../../perennial-alias/js/common/buildLocal.js' );
const { Octokit } = require( 'octokit' );
const octokit = new Octokit( {
auth: buildLocal.phetDevGitHubAccessToken
Expand Down
2 changes: 1 addition & 1 deletion github-labels/printGithubAuthorization.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// Copyright 2020, University of Colorado Boulder
// @author Michael Kauzmann (PhET Interactive Simulations)

const buildLocal = require( '../../perennial/js/common/buildLocal' );
const buildLocal = require( '../../perennial/js/common/buildLocal.js' );

console.log( `${buildLocal.developerGithubUsername}:${buildLocal.developerGithubAccessToken}` );
4 changes: 2 additions & 2 deletions github-labels/update-repos-list.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
// a newline separated list of all repos in the phetsims organization.
// @author Matt Pennington (PhET Interactive Simulations)

const axios = require( '../../perennial/node_modules/axios' );
const buildLocal = require( '../../perennial/js/common/buildLocal' );
const axios = require( '../../perennial/node_modules/axios.js' );
const buildLocal = require( '../../perennial/js/common/buildLocal.js' );
const fs = require( 'fs' );

( async () => {
Expand Down
4 changes: 2 additions & 2 deletions project-management/reportEpicIssuesNotInSpreadsheet.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@
* @author Michael Kauzmann (PhET Interactive Simulations)
*/

const _ = require( '../../perennial/node_modules/lodash' );
const buildLocal = require( '../../perennial/js/common/buildLocal' );
const _ = require( '../../perennial/node_modules/lodash.js' );
const buildLocal = require( '../../perennial/js/common/buildLocal.js' );
const https = require( 'https' );

const linksInSpreadsheet = [];
Expand Down
2 changes: 1 addition & 1 deletion sim-info/areAllReposInFile.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
*
* @author Michael Kauzmann (PhET Interactive Simulations)
*/
const getAllRepos = require( './getAllRepos' );
const getAllRepos = require( './getAllRepos.js' );
const fs = require( 'fs' );

( async () => {
Expand Down
2 changes: 1 addition & 1 deletion sim-info/getAllRepos.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
* @author Michael Kauzmann (PhET Interactive Simulations)
*/

const buildLocal = require( '../../perennial/js/common/buildLocal' );
const buildLocal = require( '../../perennial/js/common/buildLocal.js' );
const https = require( 'https' );

const getSomeRepos = async pageNumber => {
Expand Down

0 comments on commit 56535e7

Please sign in to comment.