Skip to content

Commit 9a3176d

Browse files
erik-kroghesbena
andauthored
Apply suggestions from code review
Co-Authored-By: Esben Sparre Andreasen <esbena@github.com>
1 parent f2548aa commit 9a3176d

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

  • javascript/ql/src/semmle/javascript/frameworks

javascript/ql/src/semmle/javascript/frameworks/Files.qll

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import javascript
77
/**
88
* A call that can produce a file name.
99
*/
10-
abstract private class ProducesFileNames extends DataFlow::Node {
10+
abstract private class FileNameProducer extends DataFlow::Node {
1111
/**
1212
* Gets a file name produced by this producer.
1313
*/
@@ -185,11 +185,11 @@ private class RecursiveReadDir extends FileSystemAccess, ProducesFileNames, Data
185185
}
186186

187187
/**
188-
* Models for the NodeJS library `jsonfile`.
188+
* Classes and predicates for modelling the `jsonfile` library (https://www.npmjs.com/package/jsonfile).
189189
*/
190190
private module JSONFile {
191191
/**
192-
* Reading a JSON file using the `jsonfile` library.
192+
* A reader for JSON files.
193193
*/
194194
class JSONFileReader extends FileSystemReadAccess, DataFlow::CallNode {
195195
JSONFileReader() {
@@ -208,7 +208,7 @@ private module JSONFile {
208208
}
209209

210210
/**
211-
* Writing a JSON file using the `jsonfile` library.
211+
* A writer for JSON files.
212212
*/
213213
class JSONFileWriter extends FileSystemWriteAccess, DataFlow::CallNode {
214214
JSONFileWriter() {

0 commit comments

Comments
 (0)