Skip to content

Commit

Permalink
chore: remove obsolete IE11 code
Browse files Browse the repository at this point in the history
When we upgraed to @sinonjs/eslint-config@4 in
09e99f4 we effectively removed support
for IE11.

That means that this code branch is obsolete and is safe to remove.
  • Loading branch information
mroderick committed Jan 20, 2024
1 parent 5010ffe commit f82068f
Showing 1 changed file with 2 additions and 15 deletions.
17 changes: 2 additions & 15 deletions lib/class-name.js
Original file line number Diff line number Diff line change
@@ -1,26 +1,13 @@
"use strict";

var functionName = require("./function-name");

/**
* Returns a display name for a value from a constructor

Check warning on line 4 in lib/class-name.js

View workflow job for this annotation

GitHub Actions / lint

There must be a newline after the description of the JSDoc block
* @param {object} value A value to examine
* @returns {(string|null)} A string or null
*/
function className(value) {
return (
(value.constructor && value.constructor.name) ||
// The next branch is for IE11 support only:
// Because the name property is not set on the prototype
// of the Function object, we finally try to grab the
// name from its definition. This will never be reached
// in node, so we are not able to test this properly.
// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/name
(typeof value.constructor === "function" &&
/* istanbul ignore next */
functionName(value.constructor)) ||
null
);
const name = value.constructor && value.constructor.name;
return name || null;
}

module.exports = className;

0 comments on commit f82068f

Please sign in to comment.