Skip to content

prevent concatenation of rake lint params to npm lint params#299

Merged
justin808 merged 1 commit intoshakacode:masterfrom
mpugach:fix_rake_linters
Feb 27, 2016
Merged

prevent concatenation of rake lint params to npm lint params#299
justin808 merged 1 commit intoshakacode:masterfrom
mpugach:fix_rake_linters

Conversation

@mpugach
Copy link
Copy Markdown
Contributor

@mpugach mpugach commented Feb 26, 2016

Review on Reviewable

if (err) console.error(err);
console.log(
'=> 🔥 Webpack development server is running on port ' + PORT
`=> 🔥 Webpack development server is running on port ${PORT}`
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

now this is spotted by linters

@justin808
Copy link
Copy Markdown
Member

:lgtm: Once we pass CI, I'll merge. BIG Thanks @mpugach!


Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.


lib/generators/react_on_rails/templates/base/base/client/server.js, line 67 [r1] (raw file):
👍


Comments from the review on Reviewable.io

justin808 added a commit that referenced this pull request Feb 27, 2016
prevent concatenation of rake lint params to npm lint params
@justin808 justin808 merged commit 5cb8dbb into shakacode:master Feb 27, 2016
@justin808
Copy link
Copy Markdown
Member

Merged. Thanks @mpugach!

AbanoubGhadban pushed a commit that referenced this pull request Sep 25, 2025
Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.0.4 to 3.1.2.
- [Release notes](https://github.com/isaacs/minimatch/releases)
- [Commits](isaacs/minimatch@v3.0.4...v3.1.2)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
AbanoubGhadban pushed a commit that referenced this pull request Sep 26, 2025
Bumps [minimatch](https://github.com/isaacs/minimatch) from 3.0.4 to 3.1.2.
- [Release notes](https://github.com/isaacs/minimatch/releases)
- [Commits](isaacs/minimatch@v3.0.4...v3.1.2)

---
updated-dependencies:
- dependency-name: minimatch
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants