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

chore: manual eslint fixes #9750

Merged
merged 10 commits into from
Nov 6, 2018
Merged

chore: manual eslint fixes #9750

merged 10 commits into from
Nov 6, 2018

Conversation

pieh
Copy link
Contributor

@pieh pieh commented Nov 6, 2018

this fixes problems that eslint --fix can't autofix

@pieh pieh requested review from a team as code owners November 6, 2018 19:42
@pieh
Copy link
Contributor Author

pieh commented Nov 6, 2018

@lipis lint is green! :D still some warning, but I left them for future to not delay PR


class CommunityHeader extends Component {
render() {
const { forHire, hiring, submissionText } = this.props
const { /*forHire, hiring,*/ submissionText } = this.props
Copy link
Contributor Author

Choose a reason for hiding this comment

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

decided to comment out "unused vars" as rest of the code that was using that is commented - presumably can be used in future?

Copy link
Contributor

Choose a reason for hiding this comment

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

commented out code should be removed in general from master.

Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

Looks good, left a few comments!

@@ -1,4 +1,6 @@
#!/usr/bin/env node
const util = require(`util`)
const exec = util.promisify(require(`child_process`).exec)
Copy link
Contributor

Choose a reason for hiding this comment

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

This should fix some things 🎉

www/src/views/starter-library/starter-list.js Outdated Show resolved Hide resolved
Copy link
Contributor

@lipis lipis left a comment

Choose a reason for hiding this comment

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

now it's up to the robots

Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

🎉

@pieh pieh merged commit 7625e8e into gatsbyjs:master Nov 6, 2018
@pieh pieh deleted the manual-eslint-fixes branch November 6, 2018 20:08
gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
* gatsbygram: consistent return

* using-faker: fix linting

* using-gatsby-source-graphql: import graphql from gatsby

* using-remark-copy-linked-files: consistent return

* gatsby-source-contentful: no-self-assign

* cache-dir: consistent return, unused vars

* add-npm-owner: re-add missing imports

* fix www linting

* gatsbygram: remove unused lifecycle methods

* return null instead of fragment
@pieh pieh restored the manual-eslint-fixes branch May 17, 2020 20:35
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.

4 participants