refactoring of the fs module.

Open
#279 32 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
25/100
Issue type
Refactor
Clarity
Needs clarification
Activity status
Stale
Tech stack
c, javascript

Research direction

Start by reviewing the fs module and the referenced commit 1e9f7f506472cb7f1ae218681b319bdd279bb1ec. Trace where arguments containing path strings are handled, then verify that this handling has one shared location without changing fs behavior.

Written by the indexing model from the issue text.

Description

fs help wanted njs refactoring WIP
  1. Handle arguments containing path strings in one place:
    https://github.com/nginx/njs/commit/1e9f7f506472cb7f1ae218681b319bdd279bb1ec
Dominant language
C
Stars
1.6k
Forks
245
Avg merge
2d 5h
Merged PRs (30d)
6

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from nginx/njs

All issues in nginx/njs

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.