Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

RC > Replace JSCS/JSHint with ESLint #506

Merged
merged 1 commit into from
Jan 4, 2019
Merged

Conversation

Blackbaud-SteveBrush
Copy link
Member

No description provided.

@codecov-io
Copy link

codecov-io commented Jan 3, 2019

Codecov Report

Merging #506 into rc-2.0.0 will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           rc-2.0.0   #506   +/-   ##
=======================================
  Coverage       100%   100%           
=======================================
  Files            54     54           
  Lines          1709   1707    -2     
  Branches        249    249           
=======================================
- Hits           1709   1707    -2
Flag Coverage Δ
#builder 100% <100%> (ø) ⬆️
#runtime 100% <ø> (ø) ⬆️
#srcapp 100% <ø> (ø) ⬆️
Impacted Files Coverage Δ
config/webpack/common.webpack.config.js 100% <ø> (ø) ⬆️
cli/pact.js 100% <ø> (ø) ⬆️
cli/generate.js 100% <ø> (ø) ⬆️
loader/sky-pages-module/index.js 100% <ø> (ø) ⬆️
config/karma/pact.karma.conf.js 100% <ø> (ø) ⬆️
lib/sky-pages-route-generator.js 100% <100%> (ø) ⬆️
lib/locale-assets-processor.js 100% <100%> (ø) ⬆️
utils/host-utils.js 100% <100%> (ø) ⬆️
cli/build-public-library.js 100% <100%> (ø) ⬆️
config/webpack/alias-builder.js 100% <100%> (ø) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37aee93...ded51d5. Read the comment docs.

const libFiles = libPaths.reduce((accumulator, libPath) =>
accumulator.concat(
const libFiles = libPaths.reduce((accumulator, libPath) => {
return accumulator.concat(

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does adding return here change the functionality at all?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Never mind. I just realized you added curly braces around it so that adding return makes it equivalent to the previous code. I'm curious as to what rule the original code broke, though.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, since the previous version is returning the value of accumulator.concat implicitly. ESlint was complaining about the format of the arrow function, so writing it out explicitly shouldn't affect the end result.

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 4888bab into rc-2.0.0 Jan 4, 2019
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the rc-eslint branch January 4, 2019 16:49
Blackbaud-DiHuynh pushed a commit to Blackbaud-DiHuynh/skyux-builder that referenced this pull request Jul 10, 2020
* RC > Removed all references to SKY UX, changed dependency structure (blackbaud#419)

* RC > Updated TSLint rules (blackbaud#422)

* Removed legacy omnibar (blackbaud#420)

* RC > Config params as an object; always decode URL params (blackbaud#421)

* RC > Always provide `envId` regardless of permission scope (blackbaud#427)

* RC > Allow SPAs to bundle stylesheets that live outside of `src/app` (blackbaud#428)

* RC > Adjusted dev dependencies (blackbaud#429)

* RC > Fixed ts-helpers for build (blackbaud#434)

* RC > Update from master (blackbaud#425)

* RC > Removed global RxJS imports (blackbaud#438)

* RC > Replaced error component with iframe (blackbaud#436)

* RC > Removed SKY CSS import (blackbaud#443)

* RC > Instrument different directory for libraries (blackbaud#448)

* RC > Do not ignore Protractor Error 199 (blackbaud#435)

* RC > Merged master (blackbaud#444)

* RC > Merge master (blackbaud#454)

* RC > Upgrade Angular, RxJS, TypeScript (blackbaud#495)

* RC > Moved auth-client to peer dependency; fixed `skyux watch` (blackbaud#503)

* Disabled webpack host check (blackbaud#505)

* Replaced JSHint with ESLint. (blackbaud#506)

* RC > Merge master (blackbaud#508)

* RC > Changed name of NPM package (blackbaud#501)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants