Comments (6)
Yeah seems like afew people didn't run npm test when submitting, I got quite confused when i submitted my pull request and saw that I failed code check saying i submitted other people's pixels.
from open-pixel-art.
Thank you so much for opening your first issue for this project! We'll try to get back to it as quickly as possible. While you are waiting...here's a random picture of a corgi (powered by dog.ceo)
from open-pixel-art.
My PR #500 is having this issue.
from open-pixel-art.
my PR #499 with similar problem
from open-pixel-art.
I'm sorry for that inconvenience. I added a git hook to the project in #511 that will hopefully reduce this happening. At least for the people following the instructions. Maybe a GitHub Action on push would be useful that will run the linting again and will push any changes if necessary.
from open-pixel-art.
Hi!, @dkundel this problem has been fixed but needs an update in CONTRIBUTING.md to remove the npm run format:json
instruction. I will be working on that.
from open-pixel-art.
Related Issues (20)
- [TRANSLATION] Add docker section on CONTRIBUTING.md portuguese translation HOT 2
- [BUG] "npm test" fails HOT 3
- [BUG] There's no format:json script in package.json HOT 2
- [BUG] Bad license link HOT 2
- [TRANSLATION] Add French translation for contribution guide HOT 1
- [BUG] Zero coordinates doesn't show up on mouseOver. HOT 6
- [BUG] ticket 3196 merge bug HOT 1
- [TRANSLATION] Add Polish HOT 2
- Not recognizing my contribution [BUG] HOT 2
- Unfortunately the automatic code review has failed. Please click the details button for more information. If the details reveal a fail in Danger please note that there is currently a known issue (\#2046) and @dkundel will merge your PR manually in the coming days. Sorry for the inconvenience. HOT 2
- Merge bot fails to merge correctly formatted pull requests. HOT 10
- [BUG] 'npm test' times out HOT 11
- [TYPO] Says Pix instead of pick HOT 3
- [BUG] Docker Build Failing in Step 4, npm POSTInstall | Git Not Found Issue HOT 2
- [BUG] NPM TEST Failed | Merge Pixels File | Docker Build Last Step HOT 1
- [BUG] Zero coordinates doesn't show up on mouseOver HOT 4
- open-pixel-art HOT 3
- [BUG] Netlify deploy error HOT 1
- [FEATURE] Please process pull requests HOT 1
- [FEATURE] Add other contributors to work thru the pull request backlog...
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 open-pixel-art.