Comments (13)
+1
from xo.
I think this could be useful. Is there ever a scenario where you would want to lint .gitignore
d files?
from xo.
Amen. I really dislike duplicating my build
and report
(etc) directories in xo.ignore
in package.json, when they are already specified in .gitignore. There is existing precedent for looking there as the Single Source of Truth for such things. For example, npm consults .gitignore for the npm pack
procedure, which is in turn used by npm publish
and friends.
If XO automatically ignores .gitignore patterns, it will keep our configuration DRY and make projects easier to maintain.
from xo.
I like this idea too, looks totally reasonable.
from xo.
Yup, totally, just needs someone willing to do the work. I haven't had time yet.
from xo.
Possibly helpful: https://github.com/jonschlinkert/parse-gitignore
from xo.
$40 bounty because ❤️. xoxo
from xo.
I can help with this.
from xo.
Oops I didn't pay attention that @juanj was also fixing this. :D well... I made my PR anyway
from xo.
Thanks for the bounty @sholladay. You can award it to @juanj :)
from xo.
Thanks @juanj for your efforts. I'm very happy that this issue was solved. I just want to vent that I don't think this was completely fair play. Except for the '''nested .gitignores''' and '''cwd option''' requests, my single commit which was made on Sept 30 already had the same code and tests as @juanj 's 17 commits which spanned from Sept 30 to Oct 11. I believe that at least a mention by @juanj is in order about copying the tests from my PR. But again, this isn't really about ownership of code nor about distribution of bounties as it is about collaboration. I'm just happy this got solved anyway. Now I'm ready to take all your arrows on me like this :) lol and cheers
from xo.
Hi @shadiakiki1986, I hear you. And I appreciate your work. The purpose of my bounty was to contribute to the community and to encourage people like you to do so, too. I don't want that message to get lost. I will try to find a way to reward you, too. It may take me a few days to get around to it. Likely I will cc you on another issue and award a higher bounty for that then I otherwise would have, while making it obvious it should go to you. Again, thanks for your time.
from xo.
Hey @shadiakiki1986 sorry if I didn't give you credit for that. But I don't copy your test's I just make a copy of one of the existing test and adapt it to test what we need. In the end it result in a very similar test.
Anyway, wish you luck in the next bounty!
from xo.
Related Issues (20)
- [minor] Improve documentation
- Fails with eslint 8.40.0: TypeError: Cannot read properties of undefined (reading 'getAllComments') HOT 6
- Error with 'no-unsafe-*' rules in Vite project HOT 1
- Add `--max-warnings` flag (fail on warnings) HOT 1
- xo --fix: Multiple fails with TypeError: Cannot read properties of undefined (reading 'get') HOT 2
- Expected HOT 2
- Unexpected token with dynamic import with type assertion HOT 1
- In a monorepo with turborepo and typescript which is the best setup? HOT 1
- Support with reviewdog? HOT 1
- Replace native/deprecated stylistic rules with `eslint-stylistic` HOT 2
- Automatic ecmaVersion HOT 2
- Projects in a Turborepo monorepo not using workspace package.json config HOT 2
- Deprecation of Formatting Rules in ESLint v8.53 HOT 5
- Proposal: enable strict-boolean-expressions
- XO seems to not play well with Typescript's array `extends` HOT 1
- EACCESS: permission denied on ignored directory HOT 2
- Reintroduce "length" rules to catch excess HOT 3
- List typescript extensions as well for `extensions` config
- regression: `xo: false` in nested `package.json` is not supported any more HOT 9
- xo does not check for `.xo.config.js` (or any file with dot '.' prefix) HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from xo.