Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate chalk.constructor() in favor of new chalk.Instance() #322

Merged
merged 10 commits into from
Mar 12, 2019
Merged

Deprecate chalk.constructor() in favor of new chalk.Instance() #322

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
8cb6c68
Add ChalkClass
tom-sherman Jan 26, 2019
ed509bc
Add chalk factory method
tom-sherman Jan 26, 2019
f66f1c0
Update readme to include chalk.instance instead of constructor
tom-sherman Jan 26, 2019
438f326
Replace Chalk.constructor() with deprecation error
tom-sherman Jan 26, 2019
08f9e5f
Update type definitions
tom-sherman Jan 26, 2019
6768476
Add tests
tom-sherman Jan 26, 2019
cf981e6
Fix code style. Only new-cap linting errors remain
tom-sherman Jan 27, 2019
bb16265
Change to capital case Instance
tom-sherman Jan 29, 2019
39037cb
Wording changes
tom-sherman Mar 12, 2019
fc50493
Update error message
tom-sherman Mar 12, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update error message
  • Loading branch information
@tom-sherman
tom-sherman committed Mar 12, 2019
commit fc504931b4dc882efb0b8fefdd89ad88e9c0901e
2 changes: 1 addition & 1 deletion test/constructor.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ test('Chalk.constructor should throw an expected error', t => {
chalk.constructor();
});

t.is(expectedError.message, 'Chalk.constructor() is deprecated. Use new Chalk.Instance() instead.');
t.is(expectedError.message, '`chalk.constructor()` is deprecated. Use `new chalk.Instance()` instead.');

t.throws(() => {
new chalk.constructor(); // eslint-disable-line no-new
Expand Down