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.

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

eslint arrow-spacing #2970

Merged
merged 10 commits into from
Sep 29, 2017
3 changes: 2 additions & 1 deletion .eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,7 @@
}], // http://eslint.org/docs/rules/space-before-function-paren
"space-infix-ops": 2, // http://eslint.org/docs/rules/space-infix-ops
"space-in-parens": [2, "never"], // http://eslint.org/docs/rules/space-in-parens
"spaced-comment": [2, "always"] // http://eslint.org/docs/rules/spaced-comment
"spaced-comment": [2, "always"], // http://eslint.org/docs/rules/spaced-comment
"arrow-spacing": [2, { "before": true, "after": true }] // https://eslint.org/docs/rules/arrow-spacing
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class LoginButtons extends Component {
}

renderLoginButtons() {
const enabledServices = this.props.loginServices().filter((service) =>{
const enabledServices = this.props.loginServices().filter((service) => {
return service.enabled;
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ test("OrderSummary snapshot test", () => {
// Initializing all the props passed into order summary component
const dateFormat = jest.fn();
const tracking = jest.fn();
const shipmentStatus = jest.fn(()=>({}));
const printableLabels = jest.fn(()=>({}));
const shipmentStatus = jest.fn(() => ({}));
const printableLabels = jest.fn(() => ({}));
const profileShippingAddress = {};
const order = {
shipping: [{ shipmentMethod: {} }],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ class InvoiceActions extends Component {
amount: true
}
}}
onChange={(event, data)=>{
onChange={(event, data) => {
this.setState({
value: data.numberValue
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -285,7 +285,7 @@ class InvoiceContainer extends Component {
cancelButtonColor: "#98afbc",
confirmButtonText: i18next.t("order.cancelOrderNoRestock"),
cancelButtonText: i18next.t("order.cancelOrderThenRestock")
}, (isConfirm, cancel)=> {
}, (isConfirm, cancel) => {
let returnToStock;
if (isConfirm) {
returnToStock = false;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ Template.coreOrderShippingInvoice.events({
cancelButtonColor: "#98afbc",
confirmButtonText: i18next.t("order.cancelOrderNoRestock"),
cancelButtonText: i18next.t("order.cancelOrderThenRestock")
}, (isConfirm, cancel)=> {
}, (isConfirm, cancel) => {
let returnToStock;
if (isConfirm) {
returnToStock = false;
Expand Down
2 changes: 1 addition & 1 deletion imports/plugins/core/router/client/startup.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ Meteor.startup(function () {
// initBrowserRouter calls Router.initPackageRoutes which calls shopSub.ready which is reactive,
// So we have to call initBrowserRouter in a non reactive context.
// Otherwise initBrowserRouter is called twice each time a subscription becomes "ready"
Tracker.nonreactive(()=> {
Tracker.nonreactive(() => {
// Make sure we have shops before we try to make routes for them
if (Array.isArray(shops) && shops.length) {
initBrowserRouter();
Expand Down
2 changes: 1 addition & 1 deletion imports/plugins/included/shippo/server/jobs/shippo.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export default function () {
// As this is run by the Server and we don't have userId()/this.userId
// which "shippo/fetchTrackingStatusForOrders" need, we use dispatch:run-as-user
// An alternative way is https://forums.meteor.com/t/cant-set-logged-in-user-for-rest-calls/18656/3
Meteor.runAsUser(ownerId, ()=> {
Meteor.runAsUser(ownerId, () => {
Meteor.call("shippo/fetchTrackingStatusForOrders", error => {
if (error) {
job.done(error.toString(), { repeatId: true });
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ class SearchModal extends Component {
<p className="rui suggested-tags" data-i18n="search.suggestedTags">Suggested tags</p>
<div className="rui search-tags">
{this.props.tags.map((tag) => (
<span className="rui search-tag" id={tag._id} key={tag._id} onClick={()=> this.props.handleTagClick(tag._id)}>{tag.name}</span>
<span className="rui search-tag" id={tag._id} key={tag._id} onClick={() => this.props.handleTagClick(tag._id)}>{tag.name}</span>
))}
</div>
</div>
Expand Down