Comments (45)
I'm going to join it to help participants and take care of what tests and docs can be good for the first contributions. But actually, the class is just one and half hour so I think it is too short for novices.
Instead of submitting a new PR to nodejs repo, I made an alternative repo of node to give such users the first experiences of contributions in https://github.com/shigeki/node_testpr/ . It can be a training for them if they submit a fake PR such as shigeki/node_testpr#1 .
Users can choice which repo they want to work on during the course.
from code-and-learn.
I've just submitted some issues of test failures to be reserved for this course in nodejs/node#9509, nodejs/node#9510 and nodejs/node#9511
from code-and-learn.
Japan has 1 collaborator (me) and 1 core team member (shigeki)
@ronkorving is a collaborator and lives in Tokyo.
from code-and-learn.
@Trott @yosuke-furukawa Indeed :) I'm not too active in the Japanese Node community, but wouldn't mind trying harder :) My Japanese is not yet 100% what I would like it to be. In any case, I signed up for Sunday (visitor) and wouldn't mind helping with the organization next year either if that's appreciated. I'll see you on Sunday. 楽しみにしてます ^^
from code-and-learn.
@ronkorving Ya!!! please help us !!!! You can come our venue and speak to our receptionist as mentor
. you can join :)
from code-and-learn.
@ronkorving Yes! we will start at 15:45 on dots Shibuya, tokyo. https://eventdots.jp/space
from code-and-learn.
Contributing Point:
doc typo:
- doc/modules.md: line 132
lookups
, it would be betterlooks up
. - doc/cluster.md: line 147, 499 has
eg.
, it would be bettere.g.
. - doc/repl: line 6 has
includable
, it would be betterincludible
. http://wikidiff.com/includable/includible - doc/dns.md: line 263
e.g.,
=>e.g.
- doc/http.md: line 553/856
e.g.,
=>e.g.
- doc/tls.md: line 762 836 1026
e.g.,
=>e.g.
doc improvement:
- doc/url.md does not have WHATWG-URL standard link and the description.
from code-and-learn.
If you need more issues, one thing you can do is search through the test files for assert.equal()
. In almost all cases, the test could be improved by using assert.strictEqual()
instead. The one exception I can think of might be test-assert.js
where use of assert.equa()
may be necessary.
from code-and-learn.
Contributing Point:
test:
- test/parallel/test-http-status-reason-invalid-chars.js: use port 0 instead of common.PORT,
const server = http.createServer((req, res) => {
if (req.url === '/explicit') {
explicit(req, res);
} else {
implicit(req, res);
}
}).listen(common.PORT, common.mustCall(() => { // should use 0
const url = `http://localhost:${common.PORT}`; // should use server.address().port
let left = 2;
const check = common.mustCall((res) => {
left--;
assert.notEqual(res.headers['content-type'], 'text/html');
assert.notEqual(res.headers['content-type'], 'gotcha');
if (left === 0) server.close();
}, 2);
http.get(`${url}/explicit`, check).end();
http.get(`${url}/implicit`, check).end();
}));
from code-and-learn.
@Trott thank you!!! sound so good. I am currently searching contribution point on code-and-learn on nodefest.
from code-and-learn.
Here's another one: There are a number of instances of setTimeout()
where the function is called without a duration (the second argument). Node.js will treat that the same as calling it with a duration of 1
. In many of these cases, it might be better to use setImmediate()
.
Here are some of those cases. I have not reviewed them to verify that setImmediate()
is better, but that might be an appropriate exercise for a first time committer.
test/sequential/test-next-tick-error-spin.js line 43- test/parallel/test-tls-hello-parser-failure.js line 30
- test/parallel/test-stream2-writable.js line 331
from code-and-learn.
test: use port 0 instead of common.PORT
- test/parallel/test-cluster-net-send.js
server.listen(common.PORT, function() { // use 0 port
socket = net.connect(common.PORT, '127.0.0.1', socketConnected); // use server.address().port
});
- test/parallel/test-tls-connect-address-family.js
- test/parallel/test-net-socket-destroy-twice.js
from code-and-learn.
@Trott ah, that's so good.
I will list these points.
from code-and-learn.
@yosuke-furukawa I'm definitely looking forward to being there and will help out in whatever way that I can. I just want to confirm that the Code and Learn is scheduled for Saturday afternoon, correct? I want to take a little bit of time Saturday morning to sight see a little bit and I want to make sure I plan things so I won't be late to the Code and Learn :-)
from code-and-learn.
I just want to confirm that the Code and Learn is scheduled for Saturday afternoon, correct?
Yes!!! Code And Learn will begin 15:45 on Saturday, and the end time is 17:15.
And I really would like you to attend NodeDiscussion on Saturday morning, almost 90 min (11:00-12:30). This discussion is unconference style. We would like to discuss about Node.js good point/bad point/wishlist. I would like share our opinions to node core committers.
If you don't have any time on Saturday morning, it is OK, but if you have time, please come!!!!!
from code-and-learn.
@shigeki awesome!!! I will wrap up these contribution point on this Thursday.
from code-and-learn.
test/sequential/test-next-tick-error-spin.js line 43
You can remove that one from the list as I'm in the process of fixing it while refactoring a bunch of other things in the test. Sorry about that.
You can replace it, though with these tests which all have at least one instance of setTimeout()
called without a second argument, and thus are candidates for possibly using setImmediate()
instead:
- test/parallel/test-stream2-readable-wrap.js line 61
- test/parallel/test-stream2-readable-non-empty-end.js lines 19 and 34
- test/parallel/test-stream2-push.js line 116
- test/parallel/test-stream2-large-read-stall.js line 49
- test/parallel/test-stream-unshift-read-race.js line 44
- test/parallel/test-stream-unshift-empty-chunk.js line 16
- test/parallel/test-stream-transform-objectmode-falsey-value.js line 33
from code-and-learn.
@yosuke-furukawa Sounds good, I'll try to make it! :) Basically just show up before 3:45?
from code-and-learn.
@Trott thank you soooooooooooooooo much!!!!!!!!!!!!!!!!!!!!!!
from code-and-learn.
https://coverage.nodejs.org/coverage-fb05e31466ac0bad/root/internal/process/stdio.js.html
- A
pseudo-tty
test that manually fires'SIGWINCH'
and makes sure none of the properties set by _refreshSize have been changed. - Add the
stderr.destroy
/stderr.destroySoon
error checking to the test which already checks that for stdout.
from code-and-learn.
@Fishrock123
Thank you sooooooooo much !!!!! This coverage information is really helpful for us.
from code-and-learn.
All contribution point:
docs:
- doc/modules.md: line 132 lookups, it would be better looks up.
- doc/cluster.md: line 147, 499 has eg., it would be better e.g..
- doc/repl: line 6 has includable, it would be better includible. http://wikidiff.com/includable/includible
- doc/dns.md: line 263 e.g., => e.g.
- doc/http.md: line 553/856 e.g., => e.g.
- doc/tls.md: line 762 836 1026 e.g., => e.g.
doc improvement:
- doc/url.md does not have WHATWG-URL standard link and the description.
test assertion improvement:
- assert.equal would be better to use assert.strictEqual
test failure with build option :
test timing API improvement:
Use setImmediate instead of setTimeout().
- test/parallel/test-tls-hello-parser-failure.js line 30
- test/parallel/test-stream2-writable.js line 331
- test/parallel/test-stream2-readable-wrap.js line 61
- test/parallel/test-stream2-readable-non-empty-end.js lines 19 and 34
- test/parallel/test-stream2-push.js line 116
- test/parallel/test-stream2-large-read-stall.js line 49
- test/parallel/test-stream-unshift-read-race.js line 44
- test/parallel/test-stream-unshift-empty-chunk.js line 16
- test/parallel/test-stream-transform-objectmode-falsey-value.js line 33
test use port:0 instead of common.PORT :
- test/parallel/test-http-status-reason-invalid-chars.js: use port 0 instead of common.PORT,
- test/parallel/test-cluster-net-send.js
- test/parallel/test-tls-connect-address-family.js
- test/parallel/test-net-socket-destroy-twice.js
coverage improvement:
- A pseudo-tty test that manually fires 'SIGWINCH' and makes sure none of the properties set by _refreshSize have been changed.
- Add the stderr.destroy/stderr.destroySoon error checking to the test which already checks that for stdout.
https://coverage.nodejs.org/coverage-fb05e31466ac0bad/root/internal/process/stdio.js.html
from code-and-learn.
assert.deepEqual would be better to use assert.deepStrictEqual
We have a lint rule for that so you shouldn't find any examples of deepEqual
that haven't been explicitly excused as necessary exceptions (e.g., in test-assert.js
).
from code-and-learn.
@Trott OK, I updated.
from code-and-learn.
i will try to fix this doc/dns.md: line 263 e.g., => e.g.
from code-and-learn.
I will try to fix test/parallel/test-cluster-net-send.js
!
from code-and-learn.
I try to fix test: test/parallel/test-net-socket-destroy-twice.js
use port 0 instead of common.PORT
#58 (comment)
from code-and-learn.
I'm gonna try test/parallel/test-stream-transform-objectmode-falsey-value.js line 33
!
from code-and-learn.
I'll try to fix doc/http.md: line 553/856 e.g., => e.g.
from code-and-learn.
I'll try to fix test/parallel/test-http-status-reason-invalid-chars.js
: use port 0 instead of common.PORT
from code-and-learn.
I will try to fix doc/tls.md: line 762 836 1026 e.g., => e.g.
from code-and-learn.
I will try to fix doc/cluster.md: line 147, 499 has eg., it would be better e.g..
from code-and-learn.
I'll try to fix test/parallel/test-tls-connect-address-family.js
: use port 0 instead of common.PORT
from code-and-learn.
I'll try to fix doc/repl: line 6 has includable, it would be better includible.
from code-and-learn.
I'll try to fix doc/modules.md: line 132 lookups, it would be better looks up.
from code-and-learn.
I'll try to fix test/parallel/test-stream2-push
, using setImmediate() instead of setTimeout()
from code-and-learn.
I'll try to improve coverage of stdio
.
from code-and-learn.
I will try to fix test/parallel/test-stream-unshift-empty-chunk.js
from code-and-learn.
I'll try to fix test/parallel/test-stream-unshift-empty-chunk.js
from code-and-learn.
送っちゃった
https://github.com/nodejs/node/pull/9579/files
from code-and-learn.
I would like to ask for English native speakers to confirm if series of PRs to correct e.g.,
are right in English grammar.
from code-and-learn.
Good job everyone :) I had a blast this weekend. Thank you all.
from code-and-learn.
@yosuke-furukawa do you know if anyone took up the two suggestions I had?
from code-and-learn.
@Fishrock123 Unfortunately, no one choose your suggestions, everyone chooses minor typo and minor test fix. BUT I share coverage information to Japanese noders.
from code-and-learn.
Ok, just wanted to know so I could pass them on to others/do them myself. :)
from code-and-learn.
Related Issues (20)
- Code & Learn Saint Petersburg 2020 HOT 2
- Next events? HOT 10
- Mentors for Code-and-Learn @ NodeConf EU (Nov 5) HOT 8
- Scope of Code + Learn for JS Interactive 2018 HOT 8
- Code + Learn Copy for the JS Interactive Website HOT 7
- Mentors for Code + Learn @ Node+JS Interactive 2018 HOT 36
- COSCon'18 (China Open Source Conf) wants to hold a Code + Learn Workshop HOT 55
- Common questions at C&L events HOT 7
- Mentors for Code + Learn @ NodeConf Colombia 2019 HOT 14
- Bangalore, November 17th HOT 16
- Example coverage task HOT 1
- Mentors for Code-and-Learn @ Seattle (Dec 18) HOT 7
- JSConf Budapest 2019 would like to hold a Code & Learn Workshop HOT 11
- Code And Learn at NodeFest 2018 HOT 4
- CFP: Code + Learn @ JSConf.Asia (Singapore) 2019-06-[14|15|16] HOT 5
- Code And Learn At Beijing April 28th HOT 7
- Code & Learn Saint Petersburg 2019 HOT 9
- Bangalore, May 25th HOT 4
- Node.js Code + Learn at COSCon'19 (China Open Source Conf) HOT 14
- Code & Learn Moscow 2019 HOT 2
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 code-and-learn.