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

use shared settings #194

Merged
merged 1 commit into from
Dec 13, 2018
Merged

use shared settings #194

merged 1 commit into from
Dec 13, 2018

Conversation

bertho-zero
Copy link
Contributor

Fix #192

@bertho-zero
Copy link
Contributor Author

The tests break exactly the same way at home with or without the modifications.

@bertho-zero
Copy link
Contributor Author

@azz @BPScott @duailibe @zimme @zertosh @suchipi

Can anyone watch this PR please?

@zimme
Copy link
Member

zimme commented Oct 31, 2018

I can try and get to it this weekend

@docwhat
Copy link

docwhat commented Dec 11, 2018

Any progress on this?

docwhat added a commit to docwhat/website that referenced this pull request Dec 11, 2018
- jsx-eslint/eslint-plugin-react#2082 is just the symptom that is causing
my editor to add "Warnings" to the top of my .js files.

- prettier/prettier-eslint#194 is the actual bug; prettier-eslint isn't
passing the `baseConfig` to eslint.
docwhat added a commit to docwhat/website that referenced this pull request Dec 12, 2018
- jsx-eslint/eslint-plugin-react#2082 is just the symptom that is causing
my editor to add "Warnings" to the top of my .js files.

- prettier/prettier-eslint#194 is the actual bug; prettier-eslint isn't
passing the `baseConfig` to eslint.
@suchipi
Copy link
Member

suchipi commented Dec 12, 2018

I'd merge this but I don't have write access; @zimme @kentcdodds if you give me GitHub/npm access I'll take care of this

@j-f1
Copy link
Member

j-f1 commented Dec 12, 2018

@suchipi Want me to add you to the ESLint team?

@suchipi
Copy link
Member

suchipi commented Dec 12, 2018

Sure

@j-f1
Copy link
Member

j-f1 commented Dec 12, 2018

Done ✅

@suchipi suchipi merged commit e8f6256 into prettier:master Dec 13, 2018
@zimme
Copy link
Member

zimme commented Dec 13, 2018

Thanks guys for taking care of this, sorry for ghosting you on this but recently got a baby so i haven't had any time for any of my regular open-source work.

@kentcdodds
Copy link
Member

CONGRATULATIONS @zimme!!

@zimme
Copy link
Member

zimme commented Dec 13, 2018

@kentcdodds Thanks 😄

@suchipi
Copy link
Member

suchipi commented Dec 14, 2018

Congrats @zimme!!! No need to apologize, that is infinitely more important than this!

That said... 😅 @kentcdodds maybe you could help me out? Although this has been merged, it hasn't been published to npm yet. Could I get access so I can take care of it?

@kentcdodds
Copy link
Member

Hmmm... I haven't been maintaining this project in quite a while. I'm not sure whether @zimme set up automated releases or not. I'll look.

@kentcdodds
Copy link
Member

Ah, yes, releases are automated, but the build is busted with a bunch of test errors like:

Cannot find module '/home/travis/build/prettier/prettier-eslint/.prettierrc.js'

I'm not sure what's going on there 😖

@zimme
Copy link
Member

zimme commented Dec 14, 2018

Automated releases should be set up, but it seems the tests are failing so it can't publish a new release

@suchipi
Copy link
Member

suchipi commented Dec 15, 2018

Ah, okay. Then I just need to fix the tests. Thanks!

@suchipi
Copy link
Member

suchipi commented Jan 2, 2019

I took a stab at this today but the cause wasn't obvious so I lost interest :/ might try again later but if someone else does it before me, I would not at all be offended

@cjones26
Copy link

Are we not going to get a release until someone fixes the tests :)?

@zimme
Copy link
Member

zimme commented Jun 16, 2019

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@zimme zimme added the released label Jun 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warning: React version not specified in eslint-plugin-react settings, though it is specified
7 participants