Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
fix: clean up some deprecation warnings
Browse files Browse the repository at this point in the history
Closes #332
  • Loading branch information
alexlafroscia committed Jun 16, 2018
1 parent bcf1bfa commit d1990ea
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
4 changes: 3 additions & 1 deletion .eslintrc.js
Expand Up @@ -17,7 +17,9 @@ module.exports = {
browser: true
},
rules: {
'prettier/prettier': 'error'
'prettier/prettier': 'error',

'no-console': 'off'
},
overrides: [
// tests
Expand Down
10 changes: 5 additions & 5 deletions addon/mixins/ajax-request.js
Expand Up @@ -42,7 +42,7 @@ import {
import isString from 'ember-ajax/-private/utils/is-string';
import AJAXPromise from 'ember-ajax/-private/promise';

const { Logger, Test, testing } = Ember;
const { Test } = Ember;
const JSONContentType = /^application\/(?:vnd\.api\+)?json/i;

function isJSONContentType(header) {
Expand Down Expand Up @@ -97,7 +97,7 @@ function stripSlashes(path) {
}

let pendingRequestCount = 0;
if (testing) {
if (Ember.testing) {
Test.registerWaiter(function() {
return pendingRequestCount === 0;
});
Expand Down Expand Up @@ -601,10 +601,10 @@ export default Mixin.create({
} else if (typeof matcher === 'string') {
return matcher === host;
} else {
Logger.warn(
'trustedHosts only handles strings or regexes.',
console.warn(
'trustedHosts only handles strings or regexes. ',
matcher,
'is neither.'
' is neither.'
);
return false;
}
Expand Down

0 comments on commit d1990ea

Please sign in to comment.